Skip to content

feat: Registration Todos#668

Merged
kris-daxiom merged 2 commits intobcgov:mainfrom
kris-daxiom:bulk-permit-validator
Apr 8, 2025
Merged

feat: Registration Todos#668
kris-daxiom merged 2 commits intobcgov:mainfrom
kris-daxiom:bulk-permit-validator

Conversation

@kris-daxiom
Copy link
Copy Markdown
Contributor

Issue:

Description of changes:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the BC Registry and Digital Services BSD 3-Clause License

@kris-daxiom kris-daxiom requested a review from thorwolpert as a code owner April 8, 2025 00:14
@kris-daxiom kris-daxiom requested a review from dimak1 April 8, 2025 00:31
Copy link
Copy Markdown
Collaborator

@dimak1 dimak1 left a comment

Choose a reason for hiding this comment

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

Added my comments. Also I created a feature branch for renewals feature-reg-renewals, just in case if we want to merge to main once it is more stable.

Comment thread strr-api/src/strr_api/resources/registrations.py
Comment thread strr-api/src/strr_api/resources/registrations.py Outdated
@dimak1 dimak1 added API Renewals Registration Renewals labels Apr 8, 2025
@kris-daxiom
Copy link
Copy Markdown
Contributor Author

Added my comments. Also I created a feature branch for renewals feature-reg-renewals, just in case if we want to merge to main once it is more stable.

This is an independent feature from the API perspective and can go into the main directly

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Apr 8, 2025

Copy link
Copy Markdown
Collaborator

@dimak1 dimak1 left a comment

Choose a reason for hiding this comment

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

👍🏻

@kris-daxiom kris-daxiom merged commit d17ec86 into bcgov:main Apr 8, 2025
10 checks passed
kris-daxiom added a commit that referenced this pull request Apr 8, 2025
* feat: Registration Todos (#668)

* feat: Registration Todos

* Added Expired to the status array

* API - NOC Events & Created Date Update (#663)

* 26983 - events & timestamp type update

Signed-off-by: Shaanjot Gill <gill.shaanjots@gmail.com>

* 26983 - lint fix

Signed-off-by: Shaanjot Gill <gill.shaanjots@gmail.com>

* 26983 - nox_expiry unit test

Signed-off-by: Shaanjot Gill <gill.shaanjots@gmail.com>

* 26983 - lint fix

Signed-off-by: Shaanjot Gill <gill.shaanjots@gmail.com>

* 26983 - cleanup

Signed-off-by: Shaanjot Gill <gill.shaanjots@gmail.com>

* 26983 - cleanup

Signed-off-by: Shaanjot Gill <gill.shaanjots@gmail.com>

* 26983 - add UnitTestConfig to noc_expiry job conftest

Signed-off-by: Shaanjot Gill <gill.shaanjots@gmail.com>

* 26983 - isort fix

Signed-off-by: Shaanjot Gill <gill.shaanjots@gmail.com>

* 26983 - run mode fix

Signed-off-by: Shaanjot Gill <gill.shaanjots@gmail.com>

* 26983 - run mode fix

Signed-off-by: Shaanjot Gill <gill.shaanjots@gmail.com>

* 26983 - check assignee, feedback changes, remove noc_expiry job changes

Signed-off-by: Shaanjot Gill <gill.shaanjots@gmail.com>

* 26983 - lint fixes

Signed-off-by: Shaanjot Gill <gill.shaanjots@gmail.com>

---------

Signed-off-by: Shaanjot Gill <gill.shaanjots@gmail.com>

* noc expiry timezone related fix & noc events (#664)

Signed-off-by: Shaanjot Gill <gill.shaanjots@gmail.com>

* Fix scroll in NOC To Do (#667)

* Fix scroll in NOC To Do

* ci-cd (#671)

* indent (#673)

* add quote (#674)

* fix: Adding run script (#675)

---------

Signed-off-by: Shaanjot Gill <gill.shaanjots@gmail.com>
Co-authored-by: Shaanjot Gill <shaangill025@users.noreply.github.com>
Co-authored-by: Dima K <dima1086@hotmail.com>
Co-authored-by: Avni Salhotra <132402145+avni-work@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

API Renewals Registration Renewals

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants