Skip to content

Document route 202 and jubilife city scripts#933

Open
Viperio19 wants to merge 2 commits intopret:mainfrom
Viperio19:script-documentation
Open

Document route 202 and jubilife city scripts#933
Viperio19 wants to merge 2 commits intopret:mainfrom
Viperio19:script-documentation

Conversation

@Viperio19
Copy link
Contributor

This PR documents the route 202 and jubilife city script, event and text files. This does not include buildings in jubilife city to keep the PR from getting too big.

FLAG_UNK_0x01F7
FLAG_HIDE_JUBILIFE_CITY_CLOWNS_1_AND_2
FLAG_HIDE_JUBILIFE_CITY_MIDDLE_AGED_MAN
FLAG_HIDE_POKETCH_CO_1F_MIDDLE_AGED_MAN
Copy link
Collaborator

Choose a reason for hiding this comment

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

would there be value in calling these two "poketch co president" instead of "middle aged man"?

},
{
"id": "pl_msg_00000023_00010",
"id": "JubilifeCity_Text_LucasThatManWhatsHeDoing",
Copy link
Collaborator

Choose a reason for hiding this comment

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

nitpick:

Suggested change
"id": "JubilifeCity_Text_LucasThatManWhatsHeDoing",
"id": "JubilifeCity_Text_LucasThatGuyWhatsHeDoing",

},
{
"id": "pl_msg_00000023_00036",
"id": "JubilifeCity_Text_TellmeHaveYouNotObtainedAPalPad",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
"id": "JubilifeCity_Text_TellmeHaveYouNotObtainedAPalPad",
"id": "JubilifeCity_Text_TellMeHaveYouNotObtainedAPalPad",

},
{
"id": "pl_msg_00000023_00044",
"id": "JubilifeCity_Text_YoucanAddAppsToYourPoketch",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
"id": "JubilifeCity_Text_YoucanAddAppsToYourPoketch",
"id": "JubilifeCity_Text_YouCanAddAppsToYourPoketch",

FLAG_UNK_0x03F4
FLAG_UNK_0x03F5
FLAG_UNK_0x03F6
FLAG_HIDE_ROUTE_202_POKEBALL
Copy link
Collaborator

Choose a reason for hiding this comment

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

i would personally name these types of flags FLAG_OBTAINED_(LOCATION)_(ITEM). especially for later areas where there are multiple items and you need to disambiguate

ie. FLAG_OBTAINED_ROUTE_202_POTION

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done, I also documented the offsets and files for hidden/pokeball item scripts

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.

2 participants