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

Option to have MYO image be hidden or deleted #182

Open
SpeedyD opened this issue Sep 29, 2021 · 2 comments · Fixed by #215
Open

Option to have MYO image be hidden or deleted #182

SpeedyD opened this issue Sep 29, 2021 · 2 comments · Fixed by #215
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@SpeedyD
Copy link
Contributor

SpeedyD commented Sep 29, 2021

Feature Request

When approving a MYO Approval request, in the Image Settings, it currently only states "Set Active Image" and "Invalidate Old Image".
I would like to request the option to immediately hide the old image gets added, as well as the option to immediately delete the old image.
It is quite a hassle to manually delete MYO slot images for every single approved character.

@itinerare itinerare added the enhancement New feature or request label Sep 29, 2021
itinerare added a commit to itinerare/lorekeeper that referenced this issue Nov 6, 2021
- toggleable in config/lorekeeper/extensions
- adds option to hide or delete MYO slot placeholder image when approving MYO submission
@itinerare itinerare self-assigned this Nov 8, 2021
@itinerare itinerare added this to the v2.1.0 milestone Nov 8, 2021
@preimpression
Copy link
Contributor

@SpeedyD - I've been using the Develop instance of this a ton so I think this could be closed given it'll be on the next version!

@itinerare itinerare linked a pull request Apr 29, 2022 that will close this issue
@itinerare
Copy link
Collaborator

@preimpression Github automatically closes these things when the relevant content gets merged into main iirc

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants