Skip to content

Conversation

@smith113-p
Copy link
Contributor

@smith113-p smith113-p commented May 29, 2025

Bug #1868
Closes #5899
Closes #3951

@smith113-p
Copy link
Contributor Author

Unfortunately this will no longer automatically treat unmatched URLs as studio URLs:

Draft Test _ StashDB - Google Chrome 2025-05-29 15_35_01

@DogmaDragon

This comment was marked as outdated.

@smith113-p

This comment was marked as outdated.

@DogmaDragon
Copy link
Collaborator

My comment was in reference to someone using older than v0.22.0 Stash (which lacks URLs support) and trying to submit a draft. I'm not even sure if that's relevant to this change though. I just saw struct being changed. 🤷

@smith113-p
Copy link
Contributor Author

My comment was in reference to someone using older than v0.22.0 Stash (which lacks URLs support) and trying to submit a draft. I'm not even sure if that's relevant to this change though. I just saw struct being changed. 🤷

Yes this does not affect that.

@WithoutPants WithoutPants added the improvement Something needed tweaking. label Jun 2, 2025
@WithoutPants WithoutPants added this to the Version 0.29.0 milestone Jun 2, 2025
@WithoutPants WithoutPants mentioned this pull request Jun 2, 2025
@WithoutPants WithoutPants merged commit 96d2b36 into stashapp:develop Jun 3, 2025
2 checks passed
@smith113-p smith113-p deleted the urls branch June 3, 2025 00:27
smith113-p added a commit to smith113-p/stash that referenced this pull request Jun 8, 2025
The code looks like it does because it initially used
string pointers; however, the version that landed used
a regular string array, so we can just the = operator.
smith113-p added a commit to smith113-p/stash-box that referenced this pull request Jun 8, 2025
Now that Stash supports sending more than one URL at a time
(stashapp/stash#5894), this code treats
unmatched URLs as a studio URL even if multiple URLs were submitted.

This allows, for example, submitting a Clips4Sale URL and a studio
URL and having both matched.
WithoutPants pushed a commit that referenced this pull request Jun 11, 2025
The code looks like it does because it initially used string pointers; however, the version that landed used a regular string array, so we can just the = operator.
XGFan pushed a commit to XGFan/stash that referenced this pull request Oct 22, 2025
XGFan pushed a commit to XGFan/stash that referenced this pull request Oct 22, 2025
The code looks like it does because it initially used string pointers; however, the version that landed used a regular string array, so we can just the = operator.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Something needed tweaking.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature] Upon clicking the “Submit to Stash-Box” button, all urls are sent to the server. [Feature] Submit Multiple URLs with Draft

3 participants