Skip to content

Fraglink /played 1.1.0 for Retail and Forever - #30

Closed
Lucrona wants to merge 1 commit into
cursor/wow-profile-cards-4791from
cursor/wow-addon-110-4791
Closed

Lucrona wants to merge 1 commit into
cursor/wow-profile-cards-4791from
cursor/wow-addon-110-4791

Conversation

@Lucrona

@Lucrona Lucrona commented Nov 10, 2026

Copy link
Copy Markdown
Owner

Do not merge or publish. Stacked on PR 29.

Fraglink /played 1.1.0 is one addon. The same Lua file tags Retail or Forever at login. The zip has one toc (## Interface: 120105, 120107, 16001) because both clients load that file when no flavour toc is present. The separate Forever and Camelot tocs are removed.

A lower /played does not replace the total already saved on the PC. The addon never talks to Fraglink, so a refused upload cannot error in game. The saved file stays, and the same file can be uploaded again later. BNET_FOREVER_LOOKUP and BNET_FOREVER_ACCOUNT are unchanged and stay off.

CurseForge copy is in apps/desktop/addon/curseforge/ for pasting later. Nothing was published.

lua5.1 flavour and addon tests passed. luacheck reported 0 warnings. cargo fmt, clippy -D warnings, and cargo test --workspace passed.

Head 48c7c6d483d977d5a5fc364317b08a2b9501f18e.

Open in Web Open in Cursor 

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>
@Lucrona
Lucrona added this pull request to stack #31 November 10, 2026 07:13
@cursor

cursor Bot commented Nov 10, 2026

Copy link
Copy Markdown

Merged onto wow-played through #32 (merge commit 58867206db87b2395a460e1bba1e911eda924a7b). This pull request’s head is that merge’s second parent, 48c7c6d483d977d5a5fc364317b08a2b9501f18e. GitHub would not retarget this stacked pull request, so it is being closed rather than marked merged. Nothing was deployed.

@Lucrona Lucrona closed this Nov 10, 2026
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.

2 participants