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

[Feature Request]: Exclude rocket grunt toggle & exclude rocket leader toggle #1021

Merged

Conversation

Gattaca1
Copy link
Contributor

@Gattaca1 Gattaca1 commented Jun 19, 2024

Adds 2 toggles to further filter Rocket Pokemon: Exclude Rocket Leaders & Exclude Rocket Grunts.

It should be noted that I've only tested this with a limited data setup (actually just 2 pokestops, one with character ID 16, and one with character ID 41), so it could use a test with a larger set of data. I don't have a scanner set up so I'm unable to do this

@Gattaca1 Gattaca1 changed the base branch from main to develop June 19, 2024 19:31
Copy link
Collaborator

@TurtIeSocks TurtIeSocks left a comment

Choose a reason for hiding this comment

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

Overall looks nice and clean! Once you've made the one requested change I will to try get it tested for you. Nice work!

server/src/models/Pokestop.js Outdated Show resolved Hide resolved
@Boby360
Copy link

Boby360 commented Jun 19, 2024

I tested the code when it was on the main branch with success.

It would be amazing to have an equivalent of the Quest Conditions advanced filter, although I understand if it would be out of scope.
https://github.com/WatWowMap/ReactMap/blob/develop/src/components/filters/QuestConditions.jsx

@Gattaca1 Gattaca1 changed the base branch from develop to main June 20, 2024 02:03
@Gattaca1
Copy link
Contributor Author

Ready for review now!

Copy link
Collaborator

@TurtIeSocks TurtIeSocks left a comment

Choose a reason for hiding this comment

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

Working as expected and code looks good now, good job! You changed the target branch back to main though ;) switched that back to develop and I'll merge this.

@Gattaca1 Gattaca1 changed the base branch from main to develop June 21, 2024 23:03
@Gattaca1
Copy link
Contributor Author

Updated the branch back to develop

@TurtIeSocks TurtIeSocks merged commit c1be413 into WatWowMap:develop Jun 22, 2024
2 checks passed
Copy link

🎉 This PR is included in version 1.33.0-develop.9 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants