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

Gen 2 Support #349

Closed
TheNewDarkOne opened this issue Feb 18, 2017 · 24 comments
Closed

Gen 2 Support #349

TheNewDarkOne opened this issue Feb 18, 2017 · 24 comments

Comments

@TheNewDarkOne
Copy link

It kinda works if using text/themeless but can't use any filters or it stops working. hide common? hides common AND all gen 2, etc. so no filter, text theme to do gen 2

what needs though is images for gen 2 in themes, and filters to add support for handling up to 250.

@TheNewDarkOne
Copy link
Author

I was able to fix the theme problem myself. I just added images for pokemon 152-650 for good measure and app uses them just fine. All that remains is filter updates though so I can filter commons again without it filtering ALL gen 2

@johnforeland
Copy link

johnforeland commented Feb 19, 2017

I tried making a theme, but my icons keep blinking 😨
Would you mind sharing the zip?

Edit:
Oh never mind, my server was acting up.
To anyone wanting to make their own zip, make sure to zip the root folder, not the contents of the root folder.

Edit 2:
If anyone have the same error, I had to turn off caching of /raw_data only (not standard on RocketMap, I had to turn on caching of images and js and stuff manually in Nginx)

@thedruid
Copy link

thedruid commented Feb 19, 2017

TheNewDarkOne:

replace in global.h:
#define POKEMON_NUMBER 155

with:
#define POKEMON_NUMBER 255

and filter option is working again.

@TheNewDarkOne
Copy link
Author

I mean building from source sure, but the source is currently behind app store version. :\

@istornz
Copy link
Owner

istornz commented Feb 20, 2017

@TheNewDarkOne Can you share with me your theme plz ?
I will made a new update for full support

@TheNewDarkOne
Copy link
Author

TheNewDarkOne commented Feb 20, 2017

Honestly I took the updated sprites from rocketmap (https://stuff.notfurprofit.org/static02.zip) and used those for personal use.
I actually replaced your 1-150 sprites with these too since they were smaller and felt less cluttered overall.
I'm sure to include em in this though you should probably reach out to @sebastienvercammen who put the time into making them all.

@Alderon86
Copy link

@istornz This is important! Please hurry up i need this lovely notifications for new mons :D

@istornz
Copy link
Owner

istornz commented Feb 21, 2017

@Alderon86 I'm sick so I will do that ASAP.

@jorgitoh
Copy link

@istornz Hi, you ill update the ipa / source for 2gen pokemons?. Thank you for ur work :)

@istornz
Copy link
Owner

istornz commented Feb 22, 2017

@jorgitoh IPA & AppStore version 👍

@decsiteru
Copy link

@istornz hi is it possible to add the moveset support at the same time when clicking on the 'mon plz?

@TheNewDarkOne
Copy link
Author

One step at a time, i'd rather him just get the media/filters updated first, worry about features later.

@istornz
Copy link
Owner

istornz commented Feb 23, 2017

Anyone can test the new version with latest theme (don't have server to test) : https://github.com/istornz/iPokeGo/wiki/Centralization-of-all-downloadable-theme#official-rocketmap-theme

If it's ok I will publish a version on the AppStore 👍
Waiting for reply :)

@johnforeland
Copy link

Does that version support themes, though? Can't find it in the settings.

@TheNewDarkOne
Copy link
Author

I didn't test newest version since i only run app store version. however using app store version i downloaded that theme and the theme itself is fine. of course no filters cause it's app store version.

hope that helps!
img_1334

@decsiteru
Copy link

Can you upload an ipa to test it out? Sry I don't have a mac so i can't compile

@k9ordon
Copy link

k9ordon commented Feb 24, 2017

im also missing the themes settings :(
img_2017-02-24 12 31 50

@TheNewDarkOne
Copy link
Author

Far as I know the themes stuff was never merged into this branch. a few features appeared only in app store version which is why I stopped using source version.

@johnforeland
Copy link

Okay, but to answer your question, @istornz, everything works (ish). AppStore version (screenshot) works perfectly with the RocketMap sprites (albeit without filtering)

GitHub version (screenshot) works with gen 2, because I added notifications for some gen 2 commons and it took me to the location in the screenshot with no Pokèmon (because no gen 2 sprites in GitHub version). See both screenshots, same area, proving Github version works with gen 2 now, just without any sprites.

So, App Store version (with themes) just needs the following, as mentioned by @thedruid:
#define POKEMON_NUMBER 251 (Not 255 though, because 252-255 is only glitch Pokèmon and eggs, see screenshot.

@johnforeland
Copy link

I created a gen 2 support PR, if anyone wants it before an AppStore version is made / accepted by Apple. It contains new gen 1 and 2 sprites, and reduced number of Pokémon from 255 to 251. Before and after screenshot. (Pidgey and Spinarak hidden by filter, to name a few)

istornz added a commit that referenced this issue Feb 26, 2017
Add the theme functionality in order to remove copyrighted images from
project.
@istornz
Copy link
Owner

istornz commented Feb 26, 2017

Hi sorry guys I have forgotten to include the theme support 👎
Now this functionality is open-sourced !

@johnforeland Thanks for your PR, but I need to remove all copyrighted image from this repo 🤐
Can anyone test this new version ?

I will submit the AppStore version today !

@istornz
Copy link
Owner

istornz commented Feb 26, 2017

Version 2.5.1 submitted to the AppStore 👍

@TheNewDarkOne
Copy link
Author

TheNewDarkOne commented Feb 28, 2017

i did a clean build and tested all day, was good with rocketmap theme. filters and all were perfect

@istornz
Copy link
Owner

istornz commented Feb 28, 2017

@TheNewDarkOne Okay thanks for your reply, I close this issue 👍

@istornz istornz closed this as completed Feb 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

8 participants