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

Revert "Tijdzone voor BAG" #239

Closed
wants to merge 1 commit into from
Closed

Conversation

sebastic
Copy link
Contributor

This reverts commit b2a2038.

Fixes: #238

This reverts commit b2a2038.

Fixes: nlextract#238
Copy link
Member

@fsteggink fsteggink left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ik ben het oneens met deze wijzigingen. Zie de discussie bij issue #238.

@sebastic
Copy link
Contributor Author

sebastic commented Mar 1, 2018

Dat is zeer treurig, dan zal ik deze en gerelateerde changes in mijn fork bijhouden omdat ik niet al mijn scripts en database servers ga aanpassen om de Europe/Amsterdam timezone expliciet te gebruiken.

@justb4
Copy link
Contributor

justb4 commented Mar 1, 2018

Ook ik ben oneens met deze wijziging. Sorry @sebastic . Ik denk dat alle betrokkenen in discussie bij issue #238 (o.a. @borrob @fsteggink) de voors/tegens intensief bekeken en onderzocht hebben en om technische en zelfs "legal" (datum tijdstip in NL is van belang) redenen naar boven kwam dat TIMESTAMP WITH TIMEZONE de voorkeur heeft.

Copy link
Contributor

@justb4 justb4 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ook ik ben oneens met deze wijziging. Zie mijn comment: #239 (comment)

@fsteggink
Copy link
Member

Deze discussie ligt al een jaar stil en we zijn niet tot nieuwe inzichten gekomen. Het spijt me Bas.

@fsteggink fsteggink closed this Mar 11, 2019
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.

3 participants