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

UX and backend enhancement #4

Closed
wants to merge 0 commits into from
Closed

UX and backend enhancement #4

wants to merge 0 commits into from

Conversation

cd-FileX
Copy link
Contributor

@cd-FileX cd-FileX commented Dec 7, 2023

I wrote the details into the commit descriptions and code comments. Feel free to change anything.

I could also quickly add the function of scanning through all the models for field names matching as replacement of the name search, if you want.

Closes #3

@tatsumoto-ren
Copy link
Member

There are many different things going on, and they're not related to each other. Like, when you change the build script. It would be easier to review this PR if it was split into 10 or so smaller PRs. Please split if you can.

@cd-FileX
Copy link
Contributor Author

cd-FileX commented Dec 8, 2023

Alright, will split it once I got time again

@cd-FileX
Copy link
Contributor Author

Sorry that this takes so long. However, I think I will not be able to do this before mid-February.

@tatsumoto-ren
Copy link
Member

tatsumoto-ren commented Jan 15, 2024

Sorry that this takes so long. However, I think I will not be able to do this before mid-February.

I'm going to look into it and maybe refactor some pieces by myself first. Hopefully the changes will be compatible. Regardless, we're going to have to rewrite quite a bit of code to merge all the changes from your fork and keep the old features intact. There's no need to hurry.

@cd-FileX
Copy link
Contributor Author

cd-FileX commented Jan 19, 2024

Okay, thanks. I already updated some things like the page system to fully work, though, I yet just haven't applied them to the fork. For now you'd have to pick them out of https://codeberg.org/FileX/SubSearch, I guess. '^^ Sorry

@cd-FileX cd-FileX closed this Apr 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Card import duplicate on audio
2 participants