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

Add ComicVine link to the volume tags section #192

Open
wants to merge 1 commit into
base: development
Choose a base branch
from

Conversation

shawnphoffman
Copy link

Testing the waters a little bit. I'm more of a React frontend dev so I've been playing around with your setup. I link being able to go to ComicVine to view the linked volume so I added a link there just inside of the tags section.
CleanShot 2024-10-16 at 22 58 02@2x

@Casvt
Copy link
Owner

Casvt commented Oct 17, 2024

The reason I didn't implement it is because the slug isn't stored. Generating the slug is of course an option, but I'm not sure if it's fully working.

Does it work correctly with '? Does it work correctly with unicode chars? Etc.

If we are 100% sure that the conversion is done correctly in all scenarios, then I'll merge it.

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