-
Notifications
You must be signed in to change notification settings - Fork 56
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
[WIP] Tournament badges 212 #321
base: master
Are you sure you want to change the base?
Conversation
Here's the front-end implementation of the tournament badges, but I need some guidance on how the backend should be built. |
@jj-apps What can I help you with? Did you built the backend allready or are you unfamiliar with .net? |
I just realized in the backend, that we are having an issue with the tournament, that we should fix first. The tournament does not support battletags, just plain strings. So before we add the tournament badges, we should add an auto completion feature for the tournament badge and save the battletags within the tournament. Then we could update the player profile for the tournament aswell. And then the ui should work out of the box. Like in the backend, the class |
@modmoto I'm familiar with .Net and can do the backend implementation for this. Just need guidance on where in the database this should be stored (first time using MongoDB for me). Would adding BattleTag to TournamentPlayer be all we need? Then it can be returned via a join with the player profile? I suppose we would need to update the Tournament admin UI to allow for entering the btag too. |
jeah, updating the admin UI would be needed for that. But I would make it just a tiny bit more complex, in making the name field for the player a search box, like on the ranking page, that returns the small profile with name and btag. Then we can just take this battletag and name from the backend. This makes the tournament page much easier to handle and on top off that we can store the correct battletag in the backend. So we dont have to go through the old tournaments and searching the battletag by hand. I know this is a "little" more work than expected, but ultimately we wanted to do the search like that anyways and as we need that info, I think this is a good opportunity to go for it. Regarding the mongodb:
This can be easily written and more importantly we dont have to join. Like it is intended with mongo. We just load this, when someone opens a profile page and put it on the dto where you wanted it in this PR. We could also just make a separate route, to keep the objects separated (I would actually prefer that). Something like |
Just had a shower and scratch that idea with the additional collection. We can just add a route |
I think after @gabcinder2004 tournaments get merged the winners for the next round will be auto determined so no more manual action from admin and no need to write the code to link battle tags? |
But we still need battletags, right? Right now we only enter names and dont link them to the account. |
People register from ingame and their battle tag is used. Those tourneys should be generated and controled automatically by matchmaking |
Also our season finals? Because this PR is about that. |
Yes. Season finals should use Gab's tournaments as well. I don't see a reason why not use it? |
yes, good idea. I was just wondering if this might be an issue with observers and stuff like that. Or if b2w and the other streamers ar getting this. |
@jj-apps to say we have neglected PRs is an understatement seeing as this is now 11 months old 😄 but we're trying to get back on track. This is still very relevant, we're going to be adding the ability for admins to give players badges, and the automated tournaments are still going to be using these too, somehow, not sure exactly yet. do you want to update this and we can work on a backend solution, even if its only temporary or manually done. |
#212 Front end implementation of tournament badges