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

Performer urls #4958

Merged
merged 15 commits into from
Jun 18, 2024
Merged

Performer urls #4958

merged 15 commits into from
Jun 18, 2024

Conversation

WithoutPants
Copy link
Collaborator

Replaces the url, twitter and instagram fields with a single URLs field. The old fields are deprecated - scrapers using the old field will have the values populated in the URLs field.

The link button in the Performer page will now show a dropdown of all URLs. Twitter and Instragram links are still shown as separate link buttons and are not included in the URL dropdown.

Resolves #703

@WithoutPants WithoutPants added the feature Pull requests that add a new feature label Jun 11, 2024
@WithoutPants WithoutPants added this to the Version 0.27.0 milestone Jun 11, 2024
@bob123491234
Copy link
Contributor

The old Instagram and Twitter fields allowed for users to only enter a username, this does not account for that in the migration.

@QxxxGit
Copy link

QxxxGit commented Jun 11, 2024

Small thing: is the same styling of URL icons intentional? I noticed the Instagram icon is no longer pink but blue.

@WithoutPants
Copy link
Collaborator Author

The old Instagram and Twitter fields allowed for users to only enter a username, this does not account for that in the migration.

Good catch, thanks. The migration has been updated to translate strings that don't start with http[s]:// to URLs.

Small thing: is the same styling of URL icons intentional? I noticed the Instagram icon is no longer pink but blue.

Nope. This is now addressed as well. Thanks for the heads up.

@WithoutPants WithoutPants merged commit f267660 into stashapp:develop Jun 18, 2024
2 checks passed
@DogmaDragon DogmaDragon linked an issue Aug 19, 2024 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Pull requests that add a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature] More options for performers social media [Feature] Allow multiple url's for a performer
3 participants