Skip to content

Conversation

@Hidorikun
Copy link
Contributor

@Hidorikun Hidorikun commented May 6, 2024

This is a rewrite of #88 because it was faster and less error prone to do so than to solve the merge conflicts.

I would like to keep this smaller so merge conflicts don't affect it as much so I omitted the method refactoring from the first pull request with the intention of adding it in a separate pull request.

@Loran425 Loran425 added Type: Enhancement New feature or request Type: Refactor Code that needs to be restructured or cleaned up labels May 6, 2024
@gabrieljreed
Copy link
Contributor

This looks great!

@CyanVoxel CyanVoxel added this to the Alpha 9.3.x milestone May 10, 2024
@CyanVoxel CyanVoxel added the Type: Tests Tests or testing related label May 13, 2024
@Hidorikun
Copy link
Contributor Author

@CyanVoxel @Loran425 @gabrieljreed Kind reminder that this is ready to be merged whenever you see fit 😄

@CyanVoxel CyanVoxel merged commit 02bf15e into TagStudioDev:main May 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Type: Enhancement New feature or request Type: Refactor Code that needs to be restructured or cleaned up Type: Tests Tests or testing related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants