-
Notifications
You must be signed in to change notification settings - Fork 24
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
Public Release for Fall24 meta-release #148
Conversation
Change version to v1.0.0
Fix links for test definition
Fix links for test definition
Prepare API Checklist for sim swap subscription
Update readme for r1.2
Making aligned with other readme
update changelog for r1.2
update changelog
fix duplicate swap
🦙 MegaLinter status: ✅ SUCCESS
See detailed report in MegaLinter reports |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some suggestions, one mandatory change.
Please add @camaraproject/release-management_maintainers if ready for review by Release Management
documentation/API_documentation/sim-swap-subscriptions-API-Readiness-Checklist.md
Outdated
Show resolved
Hide resolved
documentation/API_documentation/sim-swap-subscriptions-API-Readiness-Checklist.md
Outdated
Show resolved
Hide resolved
documentation/API_documentation/sim-swap-subscriptions-API-Readiness-Checklist.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Herbert Damker <[email protected]>
Co-authored-by: Herbert Damker <[email protected]>
Co-authored-by: Herbert Damker <[email protected]>
Co-authored-by: Herbert Damker <[email protected]>
Co-authored-by: Herbert Damker <[email protected]>
Co-authored-by: Herbert Damker <[email protected]>
Co-authored-by: Herbert Damker <[email protected]>
…diness-Checklist.md Co-authored-by: Herbert Damker <[email protected]>
…diness-Checklist.md Co-authored-by: Herbert Damker <[email protected]>
…diness-Checklist.md Co-authored-by: Herbert Damker <[email protected]>
Co-authored-by: Herbert Damker <[email protected]>
Co-authored-by: Herbert Damker <[email protected]>
Huge thanks @hdamker for the review. |
Make linting happy
update name of the feature files
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Also correct: Given the resource "/sim-swap-subscriptions/v0/subscriptions" -> Given the resource "/sim-swap-subscriptions/v0.1/subscriptions"
|
||
Get timestamp of last MSISDN <-> IMSI pairing change for the provided phone number. | ||
# | ||
# Get timestamp of last MSISDN <-> IMSI pairing change for the provided phone number. | ||
|
||
Background: Common retrieveSimSwapDate setup | ||
Given the resource "sim-swap/v0/retrieve-date" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It must be v1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done :)
|
||
check if SIM swap has been performed during a past period | ||
# | ||
# check if SIM swap has been performed during a past period | ||
|
||
Background: Common checkSimSwap setup | ||
Given the resource "sim-swap/v0/check" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It must be v1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
documentation/API_documentation/sim-swap-subscriptions-API-Readiness-Checklist.md
Outdated
Show resolved
Hide resolved
…diness-Checklist.md Co-authored-by: Jose Luis Urien <[email protected]>
Change sim-swap/v0/check to sim-swap/v1/check
Change sim-swap/v0/retrieve-date to sim-swap/v1/retrieve-date
Chenge /sim-swap-subscriptions/v0/subscriptions to /sim-swap-subscriptions/v0.1/subscriptions
@jlurien Thanks for the review. Update done for the 3 files. |
Let's wait to decision on camaraproject/ReleaseManagement#89 to merge this |
Please see camaraproject/ReleaseManagement#89 on how to update the Test result statement in the API readiness checklist for this Fall24 meta-release. |
Taking care of the Test result statement accordingly to camaraproject/ReleaseManagement#89
@jlurien @tanjadegroot aligned for the Test Statement. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Release PR approved (on behalf of Release Management)
PR can now be merged and release r1.2 created. Please update https://wiki.camaraproject.org/display/CAM/SimSwap+API+Release+Tracking with date for M4 and link to r1.2 when it is created. Thanks
Thanks @jlurien for the review :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
What type of PR is this?
Add one of the following kinds:
What this PR does / why we need it:
Related to issue #109
Which issue(s) this PR fixes:
Fixes #
Special notes for reviewers:
Missing link for Test Statement for sim-swap
Changelog input
Additional documentation
This section can be blank.