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

Update reportdb on the fly #135

Merged
merged 10 commits into from
May 24, 2018
Merged

Conversation

jszanto
Copy link
Contributor

@jszanto jszanto commented Jul 25, 2017

This makes reportdb update automatically when there are changes in meetings/decisions/members. This solves the following problems (once the website is switched over to use reportdb directly):

The old method of generating reportdb was also left intact in case it still is needed in the future.

@jszanto jszanto requested a review from kokx July 25, 2017 18:53
@kokx
Copy link
Contributor

kokx commented Jul 26, 2017

Nice! Hoe stabiel verwacht je dat dit is? En is er een initial setup nodig?

Also, kunnen we misschien de reportdb naar een mysql database gooien hiermee?

@kokx
Copy link
Contributor

kokx commented Jul 26, 2017

Zal dit overigens later volledig reviewen. Daar heb ik nu de tijd niet voor.

@jszanto
Copy link
Contributor Author

jszanto commented Jul 26, 2017

Ik heb nog niet alle edgecases getest, maar het zou niet minder stabiel moeten zijn dan de huidige manier waarop reportdb wordt gegeneeerd. (Aangezien het dezelfde code gebruikt) Als initiële setup moet eenmalig alles gesynchroniseerd worden met generate reportdb full daarna werk alles vanzelf. Dit zou inderdaad een goed moment zijn om de reportdb om te zetten naar een MySQL database of misschien zelfs direct naar de website database.

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.

2 participants