Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Use hooks instead of overwriting #17

Merged
merged 2 commits into from
Feb 21, 2023
Merged

Conversation

alfredbuehler
Copy link
Contributor

@alfredbuehler alfredbuehler commented Feb 13, 2023

By replacing the entire dashboard template, any improvements in the original code get lost. And it might break other plugins.
I assume (!) the patch will fix #16 and probably #11.

@jajm
Copy link
Member

jajm commented Feb 13, 2023

It looks very interesting, I'll test it as soon as possible.
Do you know if it requires a particular version of Kanboard ? I don't remember these hooks being available when I first wrote this.

@alfredbuehler
Copy link
Contributor Author

Do you know if it requires a particular version of Kanboard ?

No, unfortunately not.

@jajm jajm merged commit f8ca6c2 into biblibre:master Feb 21, 2023
@jajm
Copy link
Member

jajm commented Feb 21, 2023

It works very well, thanks!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

No compatibility with kanext plugin
2 participants