Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add texture aliases #18

Closed
4 tasks done
Juknum opened this issue Apr 3, 2024 · 0 comments · Fixed by #25
Closed
4 tasks done

feat: add texture aliases #18

Juknum opened this issue Apr 3, 2024 · 0 comments · Fixed by #25
Assignees

Comments

@Juknum
Copy link
Member

Juknum commented Apr 3, 2024

Requirements:

Description:
Textures can have multiple names across multiple mods, some mods may not use the "right" way/convention to name their textures which can lead to contributors not finding their textures when selecting the texture to their contribution.

To do:

  • keep a name field, set when the texture is first added to the database
  • add an aliases field (string[]), where the texture already exist in the database but uploaded again (only add the name in the aliases array iff : different from name and not already in aliases
  • update contribution draft modal to allow aliases search.
  • remove hash from texture name in database, filepath already have it
@Juknum Juknum converted this from a draft issue Apr 3, 2024
@Juknum Juknum self-assigned this Apr 3, 2024
@Juknum Juknum moved this from To be described to Todo in Faithful Modded devlog Apr 3, 2024
@Juknum Juknum moved this from Todo to In Progress in Faithful Modded devlog Apr 7, 2024
Juknum added a commit that referenced this issue Apr 7, 2024
@Juknum Juknum linked a pull request Apr 7, 2024 that will close this issue
@Juknum Juknum closed this as completed in #25 Apr 7, 2024
Juknum added a commit that referenced this issue Apr 7, 2024
@github-project-automation github-project-automation bot moved this from In Progress to Done in Faithful Modded devlog Apr 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant