Skip to content

feat(library): add edit mod metadata#156

Merged
Crauzer merged 1 commit into
LeagueToolkit:mainfrom
danielriosr:feat-edit-metadata
May 8, 2026
Merged

feat(library): add edit mod metadata#156
Crauzer merged 1 commit into
LeagueToolkit:mainfrom
danielriosr:feat-edit-metadata

Conversation

@danielriosr

Copy link
Copy Markdown
Contributor

Allows user to right click on mods and edit metadata/information of the mod (without changing version nor author):

Menu:
image

Pop up and success message:
image

It will modify the .json files inside each mod located at:

AppData\Roaming\dev.leaguetoolkit.manager\mods

Solves this requested feature: https://github.com/orgs/LeagueToolkit/discussions/69

@Crauzer

Crauzer commented May 5, 2026

Copy link
Copy Markdown
Member

I think it would be nice if we also included the option to change the thumbnail of an imported mod. This is especially important as most existing mods on sites don't have the thumbnail included in the archive.

For this you'll likely need to add the thumbnail into the mod archive in the library.

@Crauzer Crauzer assigned danielriosr and unassigned danielriosr May 5, 2026
@Crauzer Crauzer added area: backend Rust/Tauri backend area: frontend React/TypeScript UI type: ux User experience improvements priority: high Important, next up labels May 5, 2026
@Crauzer Crauzer moved this to In Progress in LTK Manager May 5, 2026
@danielriosr

Copy link
Copy Markdown
Contributor Author

I think it would be nice if we also included the option to change the thumbnail of an imported mod. This is especially important as most existing mods on sites don't have the thumbnail included in the archive.

For this you'll likely need to add the thumbnail into the mod archive in the library.

Updated:

  • Added "Set Thumbnail": Allow users to pick a image to use as thumbail for the mod.
  • Added "Remove" function (if thumbnail already exists). Note that if original mod did have a thumbnail, removing it will revert back the original thumbnail included in the mod itself.
  • Included the already existing webp function transformation when adding the thumbnail.

When it does not have a thumbnaill:
image

When the mod do have a thumbnail:

image

@danielriosr

Copy link
Copy Markdown
Contributor Author

Is there something else I need to check on this RP? Looking forward your feedback, thx!

@Crauzer

Crauzer commented May 6, 2026

Copy link
Copy Markdown
Member

should also squash the commits into one

@danielriosr danielriosr force-pushed the feat-edit-metadata branch from b7b1de6 to f80d20e Compare May 6, 2026 18:49
@danielriosr

danielriosr commented May 6, 2026

Copy link
Copy Markdown
Contributor Author

should also squash the commits into one

Done! Squashed 3 commits (2 related to the feat itself + 1 the commit you merged related to DiagnosticReport).

Regarding the last merge, I think it requires you to resolve the conflicts due to changing the same file and same line.

@Crauzer

Crauzer commented May 7, 2026

Copy link
Copy Markdown
Member

You need to rebase your branch on top of current main so the conflicts go away. Then force push so history is clean.

@danielriosr danielriosr force-pushed the feat-edit-metadata branch from f80d20e to 4834622 Compare May 7, 2026 19:02
@danielriosr

Copy link
Copy Markdown
Contributor Author

You need to rebase your branch on top of current main so the conflicts go away. Then force push so history is clean.

Done! Thanks for the explanation

@Crauzer Crauzer merged commit e103e6c into LeagueToolkit:main May 8, 2026
5 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in LTK Manager May 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: backend Rust/Tauri backend area: frontend React/TypeScript UI priority: high Important, next up type: ux User experience improvements

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants