Skip to content
This repository has been archived by the owner on Aug 17, 2020. It is now read-only.

Gyms and battles #1669

Open
tomasek opened this issue Sep 17, 2016 · 20 comments
Open

Gyms and battles #1669

tomasek opened this issue Sep 17, 2016 · 20 comments
Assignees
Milestone

Comments

@tomasek
Copy link

tomasek commented Sep 17, 2016

I have read other issues and closed issues too. I have been following the development and i am familiarized with the coding so far, So, my question is: may i help develop Gyms and battles? (or any other area of the app, of course.)

@jakubsuchybio
Copy link
Collaborator

jakubsuchybio commented Sep 17, 2016

For sure! Any help is appreciated. First draft of gyms is already finished. You can test and look at them by uncommenting GameClient.cs:629 cca:
NearbyGyms.UpdateWith(newGyms, x => new FortDataWrapper(x), (x, y) => x.Id == y.Id);

  • For starters you could implement adding pokemon to the gym
  • Then for navigating through pokemons currently at the gym, you could maybe use Pivot navigation as is in new Pokedex already merged from Enhanced and Refactored Pokemon Inventory Page #1614
  • And that fighting as the last thing

Also first handling method for Gym is in GameClient.cs in region Gym Handling
So you make other handlers there and call other awaitable proto handlers from Fort.cs via _client.Fort.xyz

@tomasek
Copy link
Author

tomasek commented Sep 17, 2016

Thanks for all those informations and assign this to me. I have had tried the uncomment that line before and have the gyms on the map. But i haven't seen those methods from _client.Fort.xyz.
I shall work on this.

@khunpanya
Copy link

How're things going? I think I can help making (unbound) UI for the EnterGym page. Anyone already working on this?

@tomasek
Copy link
Author

tomasek commented Sep 19, 2016

well, i can say that i am coding and i am testing some parts already, but in a few days i will be able to get into more details. about the UI, i wouldn't mind some help :D

@SmailSmailko
Copy link

привіт))
коли зявляться спортивні клуби ібитви)) я непроти тестувати))
lumia 820 RAM 1 gb windows 10
дякую

@Malavos
Copy link

Malavos commented Sep 22, 2016

@SmailSmailko It's going to take a while.

Це збирається зайняти деякий час

@jakubsuchybio
Copy link
Collaborator

@tomasek So how is it going? Any new progress?

@SmailSmailko
Copy link

дякую
поскоріше б))) ато неможу дочекатися)) а так все влаштовує велике дякую за таку проведену здорову роботу))

@Malavos
Copy link

Malavos commented Sep 22, 2016

The user above (SmailSmailko) just commented out thanking you guys for your work. He said he is satisfied with our on progress work.

Just wanted to translate it to english.

@tomasek
Copy link
Author

tomasek commented Sep 22, 2016

@jakubsuchybio i am testing a few things now. So, i plan to commit something later tomorrow or early saturday. I have a few doubts that i need to ask.
Is there any API method to choose the team?
Is there any API method to do training?
So far i am developing only fights between diferente teams, training is getting pushed a bit further because of those doubts.

@jakubsuchybio
Copy link
Collaborator

jakubsuchybio commented Sep 22, 2016

Well @tomasek we are already making finishing touches before the next release. Could you create a PR with your progress with [WIP] in the title? Let's look what you already have.
Before next release I would like to see support for managing gyms. So being able to list through pokemon in gym, see their CP, their owner's names, seeing the lvl of the gym and current prestige/maxprestige.

I would do this as a first iteration PR. So we can release it. And let the fighting and training for the next release after.

I don't know if we have API support for any fighting kind. As I was looking I don't think so. But I may be wrong.

@ST-Apps
Copy link
Owner

ST-Apps commented Sep 23, 2016

@tomasek If something is missing from GameClient class just let me know and I'll tell you where to look. I've only implemented methods for the current features, so the one for choosing teams is probably missing.

@SmailSmailko
Copy link

неможу дочекатися суботи(((

@SmailSmailko
Copy link

дуже дякую вам))

@tomasek
Copy link
Author

tomasek commented Sep 24, 2016

@ST-Apps I was implementing a few methods in GameClient, related to the Gym and that came from the FortWrapper. But at this moment i need some info. Like, if is there any method to choose team, method to train pokemon in the gym instead of battleing, method to claim the gym (if empty or if the battle was won)
i think i can do a commit today with a few things changed on the current gym, @jakubsuchybio .

@ST-Apps
Copy link
Owner

ST-Apps commented Sep 25, 2016

@tomasek they're not in GameClient yet.

Choosing a team can be done using await _client.Player.SetPlayerTeam(TeamColor.Blue) (follow the same pattern of other methods in GameClient to add this.

For the other things you should look into _client.Fort, but I'm not sure that you'll find what you need.

@SmailSmailko
Copy link

а коли вийде обнова , бо неможу вже дочекатися((

@khunpanya
Copy link

I'm worrying. Do we have to rely on everything in the current api client or can we analyze the traffic and mimic any upcoming unimplemented api like buddy and, of course, battling?

@jakubsuchybio
Copy link
Collaborator

We already have API support for battling. It will just be totally hard time to wire it.

@khunpanya
Copy link

Good to hear that we already have it. So I'll continue to work on BattlePage UI. And will make a PR when it is complete enough.

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

No branches or pull requests

6 participants