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

Update package name and legacy terms. #34

Merged
merged 4 commits into from
Jun 18, 2024
Merged

Update package name and legacy terms. #34

merged 4 commits into from
Jun 18, 2024

Conversation

JSAssassin
Copy link
Contributor

@JSAssassin JSAssassin commented Jan 9, 2024

  • Updated package name - Changed from status-list-2021-test-suite to vc-bitstring-status-list-test-suite
  • Updated legacy terms:
    • Changed StatusList2021 to BitstringStatusList,
    • Changed StatusList2021Credential to BitstringStatusListCredential.
    • Changed StatusList2021Entry to BitstringStatusListEntry.
  • Changed tag to run the test from StatusList2021 to BitstringStatusList.

NOTE: This has not been tested. We do not want to merge this until we have implementations that uses the latest contexts. Leaving the PR as a draft for now. Also tag in vc-test-suite-implementations in w3c-ccg org will have to be updated when this PR is merged. There is a draft PR for that there.)

TODO:

  • Test cases will have to updated to match the normative statements in the spec.
  • Credential context URLs and Bitstring Status List will have to be updated to use latest ones. (VC data model 2.0 and the bitstring status list context with the updated terms)

- Change `StatusList2021` to `BitstringStatusList`,
- Change `StatusList2021Credential` to `BitstringStatusListCredential`.
- Change `StatusList2021Entry` to `BitstringStatusListEntry`.
Copy link
Contributor

@PatStLouis PatStLouis left a comment

Choose a reason for hiding this comment

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

I should of checked for this PR before having a go at this myself. Good thing is that what found matches this PR. I left a comment about the status list context which will need a solution.

Copy link
Contributor

Choose a reason for hiding this comment

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

Would it be useful too add this name change to the changelog?

## 1.1.1 - 2024-06-11

### Changed
- Updated `status-list-2021-test-suite` package name to
  `vc-bitstring-status-list-test-suite`.

Copy link
Contributor

Choose a reason for hiding this comment

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

What should we do about the https://w3id.org/vc/status-list/2021/v1 context url? Without upgrade to data model v2, what context is to be used with BitstringStatusList?

@PatStLouis
Copy link
Contributor

@JSAssassin any blockers to mark this PR as ready for review?

@aljones15 aljones15 marked this pull request as ready for review June 18, 2024 15:02
Copy link
Collaborator

@aljones15 aljones15 left a comment

Choose a reason for hiding this comment

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

Approving provided that @PatStLouis is happy with the changes.

@aljones15 aljones15 merged commit 252ac18 into main Jun 18, 2024
2 checks passed
@aljones15 aljones15 deleted the rename-package branch June 18, 2024 15:06
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