stash icon indicating copy to clipboard operation
stash copied to clipboard

Tag Favoriting

Open Dankonite opened this issue 1 year ago • 7 comments

added tag favoriting, still needs the filter support image

Dankonite avatar Mar 26 '24 10:03 Dankonite

Added the filter getting a wierd error tho and no idea what causes this. Not ready yet image

Dankonite avatar Mar 26 '24 11:03 Dankonite

Finally got the filter to work

Dankonite avatar Mar 26 '24 12:03 Dankonite

Thanks for the PR. Please address the failed checks.

WithoutPants avatar Mar 26 '24 23:03 WithoutPants

sorry, new to linters and such, no idea if that was the only problem or if I need to clean up something else also. I will google a proper linting tutorial if more stuff comes back messy

Dankonite avatar Mar 27 '24 04:03 Dankonite

sorry, new to linters and such, no idea if that was the only problem or if I need to clean up something else also. I will google a proper linting tutorial if more stuff comes back messy

Just look through your changes and undo the pointless ones, like changing indentation. That would be a start. For the others, go to checks and look at the details for why they're failing.

SpedNSFW avatar Mar 27 '24 15:03 SpedNSFW

so I ran yarn validate and some of the stuff that is throwing errors are things I didnt even mess with lmao image Do I need to make these changes from a diferent branch or something. I just forked off of develop maybe at a bad time or something

Dankonite avatar Mar 28 '24 03:03 Dankonite

Migration version conflicts with existing one https://github.com/stashapp/stash/blob/develop/pkg/sqlite/migrations/56_studio_favorite.up.sql.

You will need to either update the existing migration or bump the version.

DogmaDragon avatar Apr 26 '24 21:04 DogmaDragon