Skip to content

PSP-11331: Backend implementation for Notifications#5272

Merged
asanchezr merged 6 commits intobcgov:notificationsfrom
areyeslo:PSP-11331-ReminderKeyDate-Backend
Apr 6, 2026
Merged

PSP-11331: Backend implementation for Notifications#5272
asanchezr merged 6 commits intobcgov:notificationsfrom
areyeslo:PSP-11331-ReminderKeyDate-Backend

Conversation

@areyeslo
Copy link
Copy Markdown
Collaborator

@areyeslo areyeslo commented Apr 1, 2026

  • Add CRUD operations for notifications.
  • Add validation that only one file id and sub-id is added during creation:
---------------------------------------------------------------------------
| Subtype                     | File                                      |
---------------------------------------------------------------------------
| AGREEMENT_ID                | ACQUISITION_FILE_ID or DISPOSITION_FILE_ID |
| LEASE_CONSULTATION_ID       | LEASE_ID                                  |
| LEASE_RENEWAL_ID            | LEASE_ID                                  |
| NOTICE_OF_CLAIM_ID          | ACQUISITION_FILE_ID or MANAGEMENT_FILE_ID |
| INSURANCE_ID                | LEASE_ID                                  |
| EXPROP_OWNER_HISTORY_ID     | ACQUISITION_FILE_ID                       |
| TAKE_ID                     | ACQUISITION_FILE_ID                       |
---------------------------------------------------------------------------
  • Add privacy, only the notification user can retrieve, update or delete their own notification .
  • Validate that sub-ID is assigned to FileID.

@areyeslo areyeslo force-pushed the PSP-11331-ReminderKeyDate-Backend branch from 4b6bdbf to af1bdbc Compare April 1, 2026 19:01
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 1, 2026

See CodeCov Report Here: https://app.codecov.io/github/bcgov/psp/pull/5272

@asanchezr asanchezr changed the base branch from dev to notifications April 1, 2026 20:54
@asanchezr
Copy link
Copy Markdown
Collaborator

Changed the base of this PR to be notifications - we don't want to release this just yet

Copy link
Copy Markdown
Collaborator

@asanchezr asanchezr left a comment

Choose a reason for hiding this comment

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

Please see comments

@areyeslo areyeslo requested a review from asanchezr April 6, 2026 18:26
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 6, 2026

@asanchezr asanchezr merged commit b04a36d into bcgov:notifications Apr 6, 2026
4 checks passed
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