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 scene rating to scene filename parser #432

Merged
merged 13 commits into from
Apr 4, 2020

Conversation

WithoutPants
Copy link
Collaborator

Fixes #406

Adds {rating} as a parsable field to the scene filename parser. Added to v2 and v2.5 of the UI.

Also fixes up display issues in the v2.5 version of the scene filename parser page.

@WithoutPants WithoutPants added the improvement Something needed tweaking. label Apr 1, 2020
@WithoutPants WithoutPants added this to the Version 0.2.0 milestone Apr 1, 2020
@bnkai
Copy link
Collaborator

bnkai commented Apr 2, 2020

Works ok.
Just a question about intended usage.
Is using the filename parser to batch remove some ratings considered normal / acceptable ? Now i get Error: GraphQL error: strconv.Atoi: parsing "": invalid syntax when trying to unset some ratings using the filename parser (clear the rating field and hit apply).

@WithoutPants
Copy link
Collaborator Author

Fixed v2.5 so that it is possible to unset rating. Fixed parser results refreshing twice after applying changes. Fixed rating selection not being correctly disabled when checkbox was unchecked.

Also includes a change to ignore line endings in the scss files so that the formatter works properly on Windows.

@bnkai
Copy link
Collaborator

bnkai commented Apr 3, 2020

It seems to work ok now. Can confirm that double loading is gone and setting, unsetting are fine also.

@WithoutPants WithoutPants merged commit b3e8d1e into stashapp:develop Apr 4, 2020
Anon247 pushed a commit to Anon247/stash that referenced this pull request Apr 11, 2020
* Fix scene parser display issues in 2.5
* Dropdown menu presentation improvements
* Fix refresh on parser apply
* Ignore line endings on scss files
Tweeticoats pushed a commit to Tweeticoats/stash that referenced this pull request Feb 1, 2021
* Fix scene parser display issues in 2.5
* Dropdown menu presentation improvements
* Fix refresh on parser apply
* Ignore line endings on scss files
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] Parse rating in filename
2 participants