Skip to content

feat(ui): snapshot versions table, page, info widget#1017

Merged
dimak1 merged 2 commits intobcgov:mainfrom
dimak1:feat/snapshot-versions
Feb 5, 2026
Merged

feat(ui): snapshot versions table, page, info widget#1017
dimak1 merged 2 commits intobcgov:mainfrom
dimak1:feat/snapshot-versions

Conversation

@dimak1
Copy link
Collaborator

@dimak1 dimak1 commented Feb 5, 2026

Issue:

Description of changes:

  • Add Snapshots Versions table
  • Add Snapshots details view with info widget
  • Add unit tests
Screenshot 2026-02-04 at 14 05 40 Screenshot 2026-02-04 at 14 05 51

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

@dimak1 dimak1 requested a review from JazzarKarim February 5, 2026 00:15
@dimak1 dimak1 self-assigned this Feb 5, 2026
@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 5, 2026

@bcregistry-sre
Copy link
Collaborator

Temporary Url for review: https://strr-examiner-dev--pr-1017-sxvyjr8r.web.app

Comment on lines +83 to +85
<style>

</style>
Copy link
Collaborator

Choose a reason for hiding this comment

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

An empty style block here if we can remove it

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Cleaned up in next pr

const snapshotId = route.params.snapshotId as string
const resp = await getSnapshotById(registrationId, snapshotId)
activeRecord.value = resp.snapshotData // used for the data needed to render the details page
snapshotInfo.value = resp // user for data needed for snapshot info widget on details page
Copy link
Collaborator

@JazzarKarim JazzarKarim Feb 5, 2026

Choose a reason for hiding this comment

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

sorry for being nitpicky haha, a super small typo here, used instead of user

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

cleaned up in next pr, thanks for noticing

Copy link
Collaborator

@JazzarKarim JazzarKarim left a comment

Choose a reason for hiding this comment

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

Nice, LG! 👍

@dimak1 dimak1 merged commit 5d22fdd into bcgov:main Feb 5, 2026
14 checks passed
@dimak1 dimak1 deleted the feat/snapshot-versions branch February 5, 2026 21:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants