Skip to content

Comments

feat: migrate reorder site awards to inertia#4536

Draft
Chew wants to merge 7 commits intoRetroAchievements:masterfrom
Chew:feat/reorder-site-awards-inertia
Draft

feat: migrate reorder site awards to inertia#4536
Chew wants to merge 7 commits intoRetroAchievements:masterfrom
Chew:feat/reorder-site-awards-inertia

Conversation

@Chew
Copy link
Contributor

@Chew Chew commented Feb 8, 2026

A very long and tedious process.

The primary goal of this conversion is to simply 1:1 the page for now. This requires migrating a ton of code that is only used on this page to be used to be React components.

However, a handful of functions are re-used elsewhere, making these functions have tacked on methods that make it harder to rip out and use.

Specifically, RenderAward has a ton of stuff in there to let it be used extremely dynamically, at the cost of it being hard to move around when actually needed.

Ideally, large parts of the old code is rewritten solely with this in mind. There are surely more efficient ways to get the data we need.

Steps before ready:

  • Convert it all to React
  • Add tests (so many)
  • I18n
  • Figure out a good drag and drop...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant