Repository navigation
WoW profile cards, View all, manager tabs + addon 1.1.0 (PR 29 + PR 30) - #32
Merged
Merged
Conversation
Public cards keep the current layout and add class colour, a framed avatar, and a verified tick. A profile can show Retail only, Forever only, or Both. Both keeps the toggle and the opener. Hours stay on the manager, and hiding a character does not delete it. Co-authored-by: Lucrona <Lucrona@users.noreply.github.com>
The class line reads Level 80 Mage, in the class colour, on the profile and on View all. A missing level is left off. Hours stay off those pages. Co-authored-by: Lucrona <Lucrona@users.noreply.github.com>
The manager form looks the character up and saves the row. The tick still comes only from the linked Battle.net account list. The addon upload stays next to it, and an Armory add does not rewrite /played. Co-authored-by: Lucrona <Lucrona@users.noreply.github.com>
Each tab lists only that version, and Add character and Upload /played follow the tab. The rows are plain columns, with actions in a menu. Profile cards are unchanged. Co-authored-by: Lucrona <Lucrona@users.noreply.github.com>
…from cards again. The manager section is labelled Profile page display, with a View profile link, and the first-on-profile search sits there in the site's dark input. Verified Retail cards on the profile and the View all gallery open the character popup again, including after the roster toggle. Co-authored-by: Lucrona <Lucrona@users.noreply.github.com>
A single review page under design/release-overview lists what goes live with the Forever imports, rename merge and profile cards, and what is still not built. Fixture characters only. Co-authored-by: Lucrona <Lucrona@users.noreply.github.com>
The shared file tags the client at login. Flavour toc copies are dropped because one manifest listing 120105, 120107 and 16001 is what both clients load. A lower /played no longer replaces the total saved on the PC. Co-authored-by: Lucrona <Lucrona@users.noreply.github.com>
This was referenced Nov 10, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Do not deploy.
This brings the already-reviewed PR 29 and PR 30 commits onto
wow-played. PR 26 and PR 28 are already merged there. GitHub would not change the base of PR 29 or PR 30 because they are in a stack (422), and it would not open a second pull request from the same head branch. This branch is the same commit as PR 30,48c7c6d483d977d5a5fc364317b08a2b9501f18e. No file was changed.Merge with a merge commit once CI is green.