Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Quest: Convert The Fighting Fourth to IF #7116

Open
wants to merge 1 commit into
base: base
Choose a base branch
from

Conversation

zach2good
Copy link
Contributor

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 testedd 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?

I didn't see that this was already implemented (poorly) until I was 90% done and wondering why I was getting 2x Sprinters Shoes on completion...

Done using these caps: https://discord.com/channels/443544205206355968/446470824791375892/1264648183854923892

and exhaustive exploring of the CSs around the CSs I saw in the caps, etc.

@zach2good
Copy link
Contributor Author

I left the Adelbrecht script behind, since it has a lot of notes in it

@zach2good zach2good force-pushed the wotg_bastok_nation_quests_1 branch from 381fd84 to 25e2a63 Compare February 21, 2025 14:33
-- TODO: Verify the check for rations exists on retail
if hasLetter or hasRations then
-- You won't be considered a member of the Fighting Fourth until you take that blue recommendation letter to Bastok...
return quest:progressEvent(8)
Copy link
Contributor

Choose a reason for hiding this comment

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

It's my understanding that if the event doesn't progress the quest in some way, it should simply call:

Suggested change
return quest:progressEvent(8)
return quest:event(8)

return quest:progressEvent(139, 0, 0)
else
-- Greetings, civillian. The Seventh Cohors of the Republican Legion's Fourth Division is currently recruiting new troops.
return quest:progressEvent(139, 1, 23)
Copy link
Contributor

Choose a reason for hiding this comment

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

Do these 1 and 23 represent anything specific?

@zach2good zach2good added the hold On hold, pending further action/info label Feb 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hold On hold, pending further action/info
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants