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: Create Migration to Update Leaderboard ID Type #64

Closed zysim closed 2 years ago

zysim commented 2 years ago

Leaderboard's ID was initially set to long, and that fact is included in the initial migration files. As its ID type is now ulong, we'll need to run a second migration that updates its type.

To run a migration via the dotnet CLI:

$ dotnet tool restore # If you haven't installed CLI tools yet
$ dotnet ef migrations --project LeaderboardBackend add -c ApplicationContext UpdateLeaderboardIdType

Affected Functionality

As long as we don't merge in PRs that create new models yet, especially those that have a relationship with Leaderboards, we should be fine.

Other Relevant Issues

-

RageCage64 commented 2 years ago

Offline discussion lead to this no longer being required. GUIDs are only really important as an identifier for users, not really for anything else.