FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Filter raids by Pokemon (raid boss) by faith9 · Pull Request #2230 · RocketMap/RocketMap · GitHub

Filter raids by Pokemon (raid boss) - #2230

Open
faith9 wants to merge 9 commits into
RocketMap:developfrom
faith9:raid-filter-by-pokemon
Open

Filter raids by Pokemon (raid boss)#2230
faith9 wants to merge 9 commits into
RocketMap:developfrom
faith9:raid-filter-by-pokemon

Conversation

faith9 commented Aug 3, 2017
edited
Loading

Copy link
Copy Markdown

Added a new option to filter the raids by pokemon (raid boss)

Description

My first PR so please be gentle :)

Motivation and Context

A lot of people this days only search for Moltres, Lugia or Tyranitar
It's easier to only show them on the map.

How Has This Been Tested?

Tested on my local map

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • [x ] New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • [x ] My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

neskk commented Aug 4, 2017

Copy link
Copy Markdown
Contributor

Nice effort 👍 but I think that removing Gyms from map when they don't have the desired raid boss is kinda bad and the other raid filters try to avoid doing this. If you enable raid filtering you won't see Gyms being hidden, just their raids.

Copy link
Copy Markdown
Contributor

The remove is fine, it is under a if that check is gyms are disabled, the problem is that it will not work if gyms are enabled.

Also I think a multiselect like notifications or hide will be better instead of only choosing one.
And lastly if you don't want to do it multi select I think that you at least order them by id and use locale to get their names.

Copy link
Copy Markdown
Contributor

Is this dead, would be a nice feature to have?

Copy link
Copy Markdown
Contributor

I think the statement by @neskk that the other raids filters don't remove gyms is incorrect. They all do the same call to filter out a gym that this PR does -> removeGymFromMap

Example:

        if (Store.get('showActiveRaidsOnly')) {
            if (!isOngoingRaid(item.raid)) {
                removeGymFromMap(item['gym_id'])
                return true
            }
        }

faith9 commented Sep 1, 2017

Copy link
Copy Markdown
Author

I'm using it on my map, it can be applied on develop (I'll sync the branch today).
Unfortunately I lack the abilities and the time to make the changes requested above (convert the dropdown to a multiselect and find how to use locales).

If someone else finds it useful and like to take over it's fine with me.

faith9 commented Sep 1, 2017

Copy link
Copy Markdown
Author

Regarding the sorting: before today the sorting was by CP, the most powerful pokemons being the most interesting.
I have added today the beasts, that will not respect this order (but will be the most filtered pokemon, in the following months)

tomballgithub commented Sep 1, 2017
edited
Loading

Copy link
Copy Markdown
Contributor

Why not try to copy the multi selection that was done for 'notify of pokemon' or 'notify of rarity' selections?

Copy link
Copy Markdown
Contributor

I can tackle if you don't want to

faith9 commented Sep 1, 2017

Copy link
Copy Markdown
Author

Please tackle it, I have nothing against it :)

Copy link
Copy Markdown
Contributor

There's a bug somewhere, I just saw it get into a state where it would only show Raikou. If I switched to anything else include 'show all' it would just be blank. Switching back to Raikou worked again. When I cleared the local storage in my browser it went back to normal. Will try to figure out what caused it

Copy link
Copy Markdown
Contributor

Is this dead?

tomballgithub commented Mar 16, 2018 via email

Copy link
Copy Markdown
Contributor

Copy link
Copy Markdown
Contributor

The problem with this feature is it needs constant updating with the ever changing raid boss list. Unless you support all pokemon as being possible raid bosses is there a way to support dynamic changes to current raid bosses?

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

7 participants


Back | FazBrowse Home | New Git URL