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

Name Examination: Consider timezone when updating NR expiration date #18524

Closed
eve-git opened this issue Nov 9, 2023 · 5 comments
Closed

Name Examination: Consider timezone when updating NR expiration date #18524

eve-git opened this issue Nov 9, 2023 · 5 comments
Assignees
Labels
Hometeam Names Team Name Request Name Examination Team Priority1

Comments

@eve-git
Copy link
Collaborator

eve-git commented Nov 9, 2023

There is an issue encountered when updating the expiry date of an existing NR. During synchronization with Oracle, the date in the PostgreSQL "namex" database is copied to the Oracle "namesp" database without undergoing any conversion to the local time. As a consequence, the expiry date in the Oracle database is one day ahead of the expected value.

A modernization developer will need to change the code that updates existing expiry dates as follows:
select expiration_date at time zone 'America/Vancouver' from requests

More details please reference to
#17475

@eve-git
Copy link
Collaborator Author

eve-git commented Nov 9, 2023

summary_temp.xlsx

@ozamani9gh
Copy link
Collaborator

@iamjtodd i am bumping the priority up to 1 on this one, because its related to an ops ticket and related to this ticket here:
https://app.zenhub.com/workspaces/ops-60f8556e05d25b0011468870/issues/gh/bcgov-registries/ops-support/3126
and

urgent ticket 17475.

please let me know if you want to discuss

@eve-git
Copy link
Collaborator Author

eve-git commented Nov 29, 2023

I don't know how to check NR in COLIN... But I can provide you the data in Oracle db after updating NR expiration date. Let me know if I can be help.

@eve-git
Copy link
Collaborator Author

eve-git commented Nov 29, 2023

Deployed to test environment and ready to test

@oanyahuru
Copy link
Collaborator

Comments related to this ticket have been updated on #18301

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Hometeam Names Team Name Request Name Examination Team Priority1
Projects
None yet
Development

No branches or pull requests

5 participants