-
-
Notifications
You must be signed in to change notification settings - Fork 801
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
[Feature] Add fields director
and (studio) code
to scenes
#3051
[Feature] Add fields director
and (studio) code
to scenes
#3051
Conversation
This is awesome, could you also add the Director field (or maybe Photographer might be better suited) to Galleries? I know i have a bunch of favorite photographers, and no way to really track them in the stash currently, except for creating custom tags and applying those. Having a dedicated field might be a great addition for that. networks like metart, but also metadata sites like thenude also have this information, so editing the scrapers to also include/fill those fields would be possible then as well |
34cc2de
to
33f3a2c
Compare
@djbtwothousandtwenty that's a good idea but it's a bit out of scope for this PR. I'd like to keep this PR as small and simple as possible since this is my first contribution to this project. Depending on the feedback I'll keep that in mind for a second PR. |
Submitted a PR to stash-box to allow the submission of the fields |
Removing the draft status as this PR is ready to review. Just expect an error message from stash-box when submitting a scene, this will be fixed by stashapp/stash-box#544 |
Static code review looks good. Haven't tested yet. If you want this merged in the nearer term, I'd suggest splitting out the stash-box submission stuff and putting that into a separate, subsequent PR so that submissions don't fail while we wait for the equivalent stash-box updates to be pushed through to stashdb. |
07efc17
to
4327f38
Compare
Export/import isn't including the new fields. I think filtering criteria should be included for the new fields. |
@HappyAxolotl - the stash-box counterpart seems to be merged - stashapp/stash-box#544 |
@Foucoubou Thanks for the info! However, since no new version of stash-box has been released yet and the changes have not yet been deployed to stashdb afaik, I'd like to keep that separate from this PR so this can be merged for the 0.18.0 release hopefully |
c83073b
to
957ed21
Compare
Sorry this just got a bit messy but I decided to include the change for galleries to add a
Those were added for all new fields.
I've also added filters and sort criteria for all new fields. These are definitively the last changed for this PR, it's now ready for a second review |
director
and (studio) code
to scenesdirector
& (studio) code
to scenes and photographer
to galleries
The gallery changes should be in a separate PR. |
957ed21
to
9c32c1f
Compare
director
& (studio) code
to scenes and photographer
to galleriesdirector
and (studio) code
to scenes
@@ -39,6 +39,7 @@ const sortByOptions = [ | |||
"interactive", | |||
"interactive_speed", | |||
"perceptual_similarity", | |||
"code", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this should be scene_code
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The problem with ListFilterOptions is that it'll set the value as messageID. So it will either display the correct messageID but fail the query as the database field is called code
and not scene_code
or it will display the value because code
is not a valid messageID. With my very limited React knowledge I couldn't find a solution to this so I decided to remove the sort by scene code option entirely in 66043ff.
…on for scene code
Hi, thanks for this PR. Edit: |
Fixed in #3094
The stash-box submission functionality is waiting on the stashdb instance to be updated. |
The new stash-box version was released. Could you create a PR for 07efc17 now? |
@DogmaDragon thanks for the hint, I've opened #3335 for this |
General
This PR adds the fields
code
anddirector
toScene
objects in relation to #2359 to be inline with stash-box (cc @stg-annon). This is the first time I've worked with go, graphql and react, so I appreciate any feedback!Changes
code
anddirector
toScene
objectsscene_code
("Studio Code")code
anddirector
from stash-box sourcesThe fields
code
anddirector
are using the same datatypes and semantics as stash-box:Show details
Screenshots
Scene panels
SceneEditPanel
SceneDetailPanel
Scraping
Scrape studio code (`c032ebb9-22e4-440b-9453-5c50cd8e8843`):Scrape director (
7d54e4ec-33fe-467e-b642-db4990bb044f
):