PoGo-UWP icon indicating copy to clipboard operation
PoGo-UWP copied to clipboard

[WIP] Basic Gym support (without fighting)

Open jakubsuchybio opened this issue 8 years ago • 19 comments

Closes issues

  • Closes #1669
  • Closes #1744

Changes

  • [x] Refactored TeamColor in conventer
  • [x] Add missing data to the GymPageViewModel
  • [x] Wire up basic data (GymLevel, GymMaxPoints)
  • [x] Wire up List of pokemon and trainers data in the gym with FlipView navigation
  • [ ] Wire up Deploy Pokemon button with selection of pokemon and confirmation dialog
  • [x] Wire up Start battle button with message dialog with "Not implemented"
  • [x] Wire up SetPlayerTeam when Player have no team and he have >=5 Lvl with 2 MessageDialogs to choose a team and to confirm his choice

Other info

Look on the map:

Look inside the gym:

jakubsuchybio avatar Sep 24 '16 17:09 jakubsuchybio

this is cool that you got the pokemon Gym on the map and the Gym ready

WavingLawyer644 avatar Sep 24 '16 18:09 WavingLawyer644

Well thats amazing

Jehu147 avatar Sep 24 '16 20:09 Jehu147

Great work!

Rover656 avatar Sep 25 '16 08:09 Rover656

when does this update be available..!?

dvnjhn avatar Sep 25 '16 13:09 dvnjhn

@dvnjhn whenever they finish it.

Lavince avatar Sep 25 '16 13:09 Lavince

I can help with some images (Team Icons on Map, Plataforms, Fonts for Pokémon Name, etc...)

WodsonKun avatar Sep 25 '16 16:09 WodsonKun

It look so nice^^ Vypadá to skvěle, dobrá práce :)

vossik avatar Sep 25 '16 21:09 vossik

Dik, ale uprimne ja ten design nedelal :D. Na tej mape uz to existovalo jenom se to nezobrazovalo. A tadytu stranku delal Mr. khunpanya. Ten zaslouzi kredit za design. Ja to jenom napojuju na API a aby to vsechno fungovalo.

jakubsuchybio avatar Sep 25 '16 23:09 jakubsuchybio

Guys, do not freak out with @swisszeni commits. I just merged into this branch in my fork his PR, because I need it for my DeployButton wiring. After his PR gets merged I will rebase this and it won't be such a mess anymore.

jakubsuchybio avatar Sep 26 '16 01:09 jakubsuchybio

A little update:

There are some differences against official app. For example: The crown should appear only on highest CP pokemon and also on top of the top dot indicator. These dot indicators look correct, but when there is lvl 5 gym with only 3 pokemon, it will show only 3 indicators. It should show 5 indicators with first 2 blank and 3 occupied.

I will leave these flaws for some future touches by some UI guys.

One question though. Can someone tell me what that Battles Won means? Is it for the pokemon how much it defended? Is it for the player how much he/she won?

jakubsuchybio avatar Sep 26 '16 04:09 jakubsuchybio

This designated how many battles this Pokémon has won. In other words how many times it has successfully defended an attacker. For further clarification, based on personal experience, I have only seen it pop up for Pokémon that took down my last Pokémon in my party when I was attacking a gym.

Source: http://gaming.stackexchange.com/questions/277731/what-does-battles-won-refer-to

Rover656 avatar Sep 26 '16 07:09 Rover656

@jakubsuchybio just merged the other one, waiting for rebase.

ST-Apps avatar Sep 26 '16 08:09 ST-Apps

Ok so today I made some field testing.

  • When gym is empty you can deploy first pokemon with Pogo just fine, just getting into the empty gym is a little hard because of #1744
  • When gym is not empty and it is the same team as you are, you can also deploy pokemon just fine

All deploying was done by automatic selection via LINQ of highest favorite CP pokemon, that is deployable (has full health, is not deployed yet and is not a buddy)

For manual deployment I am waiting for @swisszeni until he implements Pokemon Selection from Pokemon Inventory, because he made new pokemon inventory and is also implementing this.

jakubsuchybio avatar Sep 27 '16 20:09 jakubsuchybio

It showed no issues in the checker but this code is good In my opinion.

programmer135 avatar Nov 09 '16 22:11 programmer135

  • Merged @khunpanya fixes for battles won, some bad light colors, other UI fixes for better scaling
  • Rebased to latest master

Now only the pokemon deployment is missing. Or should I make a dialog with Ok/Cancel where I ask user if he wants to deploy his highest CP favorite available pokemon? If OK, then deploy, if Cancel, then nothing?

jakubsuchybio avatar Nov 09 '16 23:11 jakubsuchybio

@jakubsuchybio sounds like a good idea

Rover656 avatar Nov 10 '16 19:11 Rover656

I have a little question, how can i test it?

adrianekk955 avatar Nov 13 '16 07:11 adrianekk955

This is not a forum. This is easy GitHub task, if you don't know it ask at Reddit or Discord

jakubsuchybio avatar Nov 13 '16 09:11 jakubsuchybio

Nice job guys! :) Dobrá práce kluci nevěděl jsem,že mezi vývojáře patří i češi :)

LuckysSeX avatar Nov 18 '16 00:11 LuckysSeX