Skip to content

Conversation

@ZenDiime
Copy link

@ZenDiime ZenDiime commented Jan 4, 2026

I affirm:

  • I understand that if I do not agree to the following points by completing the checkboxes my PR will be ignored.
  • I understand I should leave resolving conversations to the LandSandBoat team so that reviewers won't miss what was said.
  • I have read and understood the Contributing Guide and the Code of Conduct.
  • I have tested my code and the things my code has changed since the last commit in the PR and will test after any later commits.

What does this pull request do?

Changes the time required between Chocobo feeding attempts to Vanadiel day rollover. You have to wait until the next game day before you can advance the quest via feeding attempts.

Steps to test these changes

Be at least level 20. Accept the Chocobo Wounds quest from Brutus in Upper Jeuno. Attempt to feed the Chocobo. The quest will mention that the Chocobo is still terrified of you. You now must wait until Vanadiel day rollover in order for the Chocobo to accept another trade and advance to the next step. Repeat feedings attempts every hour until quest completes.

@ZenDiime ZenDiime marked this pull request as draft January 4, 2026 09:28
@slashtangent
Copy link
Contributor

You can just module the Chocobo on trade function to check quest:getVar(player, 'Timer') ~= VanadielUniqueDay() and set Timer to VanadielUniqueDay() after the trades.

The first trade will have Timer being 0 and then after it gets set to the current day which is unique.

local playerName = player:getName()
local currentDay = VanadielUniqueDay()

print(string.format(
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would suggest debug prints not exist at all, but if absolutely necessary, be behind a flag.

@ZenDiime ZenDiime force-pushed the base branch 10 times, most recently from bfc73fb to da509ba Compare January 4, 2026 21:12
@ZenDiime ZenDiime marked this pull request as ready for review January 4, 2026 21:37
@ZenDiime ZenDiime marked this pull request as draft January 4, 2026 21:59
@ZenDiime
Copy link
Author

ZenDiime commented Jan 4, 2026

You can just module the Chocobo on trade function to check quest:getVar(player, 'Timer') ~= VanadielUniqueDay() and set Timer to VanadielUniqueDay() after the trades.

The first trade will have Timer being 0 and then after it gets set to the current day which is unique.

roger that. ill change, test, and resubmit. ty

@ZenDiime ZenDiime marked this pull request as ready for review January 4, 2026 23:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants