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

[integratie] zoeken -> rss-feed #12

Closed
futtta opened this Issue Sep 19, 2017 · 3 comments

Comments

Projects
None yet
2 participants
@futtta
Owner

futtta commented Sep 19, 2017

om latere integratie makkelijker te maken (en ik denk aan boekenzoekers -> email of notificatie) :-)

@futtta

This comment has been minimized.

Show comment
Hide comment
@futtta

futtta Sep 19, 2017

Owner

als (wanneer) we dit doen, kun je je database-query en je presentatie-logica (momenteel beiden in drawPosts()) best opsplitsen waarbij je een getPostsFromDB() hebt die wordt opgeroepen door drawPosts() and drawRss(). ofzo .. ;-)

Owner

futtta commented Sep 19, 2017

als (wanneer) we dit doen, kun je je database-query en je presentatie-logica (momenteel beiden in drawPosts()) best opsplitsen waarbij je een getPostsFromDB() hebt die wordt opgeroepen door drawPosts() and drawRss(). ofzo .. ;-)

@futtta futtta self-assigned this Sep 21, 2017

@futtta

This comment has been minimized.

Show comment
Hide comment
@futtta

futtta Sep 21, 2017

Owner

kon me niet houden, cfr. deze commit :-)

kuis gerust op waar nodig (en voeg docblocks enzo toe)

Owner

futtta commented Sep 21, 2017

kon me niet houden, cfr. deze commit :-)

kuis gerust op waar nodig (en voeg docblocks enzo toe)

@thunderbug

This comment has been minimized.

Show comment
Hide comment
@thunderbug

thunderbug Sep 21, 2017

Collaborator

Toch al wat minder dat ik moet doen :)

Collaborator

thunderbug commented Sep 21, 2017

Toch al wat minder dat ik moet doen :)

@futtta futtta closed this Sep 25, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment