-
Notifications
You must be signed in to change notification settings - Fork 87
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
Modernization (Slim 3, Bootstrap 4, working search) #200
base: live
Are you sure you want to change the base?
Conversation
fetch results before initializing typeahead as it only needs to be fetched once
@Findus23 how far are you here? Is it possible to merge this? |
Maybe we merge some things now, and add more things in other PRs? To not have one big PR in the end, to minimize risk of breaking things, and to deploy things sooner... |
I'd say this is mostly finished. In the next days winter holidays start and I'll have more time to look into it again.
|
same here @Findus23 are you still working on this? Can we do anything? |
This would need a bit more work. |
@Findus23 is there a chance you could schedule this work over the next few weeks maybe? |
replaces #198
fixes #23 and fixes #97 and fixes #27
I had more time than expected, so I went quicker than expected.
Changes
change the logic of the searchsend data with JSON mime-typefetch it only once before initializing the searchbut also cache the /data/documents route/receive-commit-hook?token=changeme
TODO
Instance with default settings:
https://issues.lw1.at/