search

leaderboardsgg / leaderboard-backend

The backend for Leaderboards.gg. Also holds public-facing APIs.
https://info.leaderboards.gg
GNU General Public License v3.0
12 stars 16 forks source link

Task: Add platforms and socials to leaderboards #209

Open zysim opened 6 months ago

zysim commented 6 months ago

Frontend displays which platforms a game runs on, as well as social links to a game's community. We don't currently have that in our DB. Refer to https://github.com/leaderboardsgg/leaderboard-site/issues/546

Affected Functionality

Frontend components + tests will need to be updated to accommodate the fact platforms and socials are tied to the model.

Other Relevant Issues

https://github.com/leaderboardsgg/leaderboard-site/issues/546

TheTedder commented 6 months ago

I'd like to punt this set of features. See also: game categorisation, tagging, grouping.

zysim commented 6 months ago

Sorry but what do you mean by punting? Do you mean to say we're delaying these features?

TheTedder commented 6 months ago

Sorry but what do you mean by punting? Do you mean to say we're delaying these features?

That's my suggestion, yes. I don't see them as important enough to add to our set of MVP features. Links to community hubs and resources can go in the game info section for now.

zysim commented 6 months ago

We don't have a game info section, though? Going by the Figma.

I'll keep the affected parts of the LeaderboardInfo component stubbed out for now. I don't want to remove things from the component only to re-add them later.

TheTedder commented 6 months ago

We don't have a game info section, though? Going by the Figma.

I'll keep the affected parts of the LeaderboardInfo component stubbed out for now. I don't want to remove things from the component only to re-add them later.

The figma is outdated and probably shouldn't be trusted as a source of information on what features we do or don't have. Refer to the MVP features document on youtrack instead.

zysim commented 6 months ago

Yeah you're right. I'll do that moving forward