From eb01915d6ba1f7fa8152ad4ba3364e707599cf72 Mon Sep 17 00:00:00 2001 From: Dean <69436131+emdeann@users.noreply.github.com> Date: Tue, 8 Apr 2025 00:34:36 -0700 Subject: [PATCH 1/4] Update code tags --- CONTRIBUTING.md | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 7bdca528c77..c47c05e9a93 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -127,7 +127,7 @@ fetching translated text is as follows: 2. The game looks up the key in the corresponding JSON file in the user's language, e.g. - ```json + ```ts // from "en/file-name.json"... "keyName": "{{arg1}}! This is {{arg2}} of translated text!" ``` @@ -136,7 +136,7 @@ corresponding English key (in the case of LATAM Spanish, it will first default t 3. The game shows the text to the user: - ```json + ```ts "Hello! This is an example of translated text!" ``` ### Adding Translated Text @@ -163,4 +163,4 @@ It'll be merged into the locales repository after any necessary corrections, at > Some issues may require you to have unlocks on your save file which go beyond overrides. For this case, there is a save file with everything unlocked (some of which are not legitimately obtainable). 1. Start the game up locally and navigate to `Menu -> Manage Data -> Import Data` -2. Select [everything.prsv](test/testUtils/saves/everything.prsv) (`test/testUtils/saves/everything.prsv`) and confirm \ No newline at end of file +2. Select [everything.prsv](test/testUtils/saves/everything.prsv) (`test/testUtils/saves/everything.prsv`) and confirm From 2969870981620f2dde6e338d9b0143ae1a17b97e Mon Sep 17 00:00:00 2001 From: Dean <69436131+emdeann@users.noreply.github.com> Date: Tue, 8 Apr 2025 00:39:14 -0700 Subject: [PATCH 2/4] Fix missing space --- CONTRIBUTING.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index c47c05e9a93..cd5e9e7b605 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -145,7 +145,7 @@ corresponding English key (in the case of LATAM Spanish, it will first default t The workflow is: 1. Make a pull request to the main repository for your new feature. -If this feature requires new text, the text should be integrated intothe code with a new i18next key pointing to where you plan to add it into the pokerogue-locales repository. +If this feature requires new text, the text should be integrated into the code with a new i18next key pointing to where you plan to add it into the pokerogue-locales repository. 2. Make another pull request -- this time to the [pokerogue-locales](https://github.com/pagefaultgames/pokerogue-locales) repository -- adding a new entry to the English locale with text for each key From 3957ea65d39f7c9131aba6ee3ce90e58c5883c62 Mon Sep 17 00:00:00 2001 From: Dean <69436131+emdeann@users.noreply.github.com> Date: Tue, 8 Apr 2025 00:41:38 -0700 Subject: [PATCH 3/4] Fix pokecorpus link --- CONTRIBUTING.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index cd5e9e7b605..f911e27fc80 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -152,7 +152,7 @@ repository -- adding a new entry to the English locale with text for each key you added to your main PR. You *only* need to add the English key and value - the translation team will handle the rest. 3. If your feature is pulled from the mainline Pokémon games (e.g. a Move or Ability implementation), add a source link for any added text within the locale PR. -(Poké Corpus)[https://abcboy101.github.io/poke-corpus/] is a great resource for finding text from the latest mainline games; otherwise, a YouTube video link showing the text in mainline is sufficient. +[Poké Corpus](https://abcboy101.github.io/poke-corpus) is a great resource for finding text from the latest mainline games; otherwise, a YouTube video link showing the text in mainline is sufficient. 4. Ping @lugiadrien in **#dev-corner** or the current callout thread to make sure your locales PR is seen. It'll be merged into the locales repository after any necessary corrections, at which point you can test it in your main PR (after updating locales from remote) From c6b0bf4262b9279cb290201b489cb2f78e17f0ef Mon Sep 17 00:00:00 2001 From: Dean <69436131+emdeann@users.noreply.github.com> Date: Sun, 1 Jun 2025 19:36:50 -0700 Subject: [PATCH 4/4] Apply suggestions from code review Co-authored-by: NightKev <34855794+DayKev@users.noreply.github.com> --- CONTRIBUTING.md | 20 ++++++++++---------- README.md | 4 ++++ 2 files changed, 14 insertions(+), 10 deletions(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index f911e27fc80..3e13fec494d 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -16,7 +16,7 @@ Please make sure you understand everything relevant to your changes from the [Ta ## 🛠️ Development Basics -PokéRogue is built with Typescript, using the Phaser game framework. +PokéRogue is built with [Typescript](https://www.typescriptlang.org/docs/handbook/intro.html), using the [Phaser](https://github.com/phaserjs/phaser) game framework. If you have the motivation and experience with Typescript/Javascript (or are willing to learn) you can contribute by forking the repository and making pull requests with contributions. @@ -50,10 +50,10 @@ Tests show you both how things are supposed to work and the expected "flow" to g Once you have your feet under you, check out the [Issues](https://github.com/pagefaultgames/pokerogue/issues) page to see how you can help us! Most issues are bugs and are labeled with their area, such as `Move`, `Ability`, `UI/UX`, etc. There are also priority labels: -- P0: Completely gamebreaking (very rare) -- P1: Major - Game crash -- P2: Minor - Incorrect (but non-crashing) move/ability/interaction -- P3: No gameplay impact - typo, minor graphical error, etc. +- `P0`: Completely gamebreaking (very rare) +- `P1`: Major - Game crash +- `P2`: Minor - Incorrect (but non-crashing) move/ability/interaction +- `P3`: No gameplay impact - typo, minor graphical error, etc. Also under issues, you can take a look at the [List of Partial / Unimplemented Moves and Abilities](https://github.com/pagefaultgames/pokerogue/issues/3503) and the [Bug Board](https://github.com/orgs/pagefaultgames/projects/3) (the latter is essentially the same as the issues page but easier to work with). @@ -76,13 +76,13 @@ You've just made a change - how can you check if it works? You have two areas to > This will likely be your first stop. After making a change, you'll want to spin the game up and make sure everything is as you expect. To do this, you will need a way to manipulate the game to produce the situation you're looking to test. In the `src/overrides.ts` file there are overrides for most values you'll need to change for testing, controlled through the `overrides` object. -For example, here is how you could test a scenario where the player Pokemon has Protean and the enemy Pokemon has Pixilate: +For example, here is how you could test a scenario where the player Pokemon has the ability Drought and the enemy Pokemon has the move Water Gun: ```typescript const overrides = { - ABILITY_OVERRIDE: Abilities.PROTEAN, - PASSIVE_ABILITY_OVERRIDE: Abilities.PIXILATE, -} + ABILITY_OVERRIDE: Abilities.DROUGHT, + OPP_MOVESET_OVERRIDE: Moves.WATER_GUN, +} satisfies Partial>; ``` Read through `src/overrides.ts` file to find the override that fits your needs - there are a lot of them! @@ -112,7 +112,7 @@ The project intends for all text to be localized. That is, strings are pulled fr To fetch translations when you first start development in your fork or to update them on your local branch, run: ```bash -npm run update-locales:remote +git submodule update --progress --init --recursive ``` ### How Localizations Work diff --git a/README.md b/README.md index 165a36695df..1bb8c7772f3 100644 --- a/README.md +++ b/README.md @@ -2,6 +2,10 @@ PokéRogue is a browser based Pokémon fangame heavily inspired by the roguelite genre. Battle endlessly while gathering stacking items, exploring many different biomes, fighting trainers, bosses, and more! +# Contributing + +See [CONTRIBUTING.md](./CONTRIBUTING.md), this includes instructions on how to set up the game locally. + # 📝 Credits > > If this project contains assets you have produced and you do not see your name, **please** reach out, either [here on GitHub](https://github.com/pagefaultgames/pokerogue/issues/new) or via [Discord](https://discord.gg/pokerogue).