-
Notifications
You must be signed in to change notification settings - Fork 56
fix(sandbox): update welcome message and fix trial duration #716
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
Conversation
Missing ChangesetsThe following package(s) are changed by this PR but do not have a changeset:
See CONTRIBUTING.md for more information about how to add changesets. Changed Packages
|
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.
Thanks for this PR @mfrancisc
Have added some comments. PTAL :)
workspaces/sandbox/plugins/sandbox/src/components/SandboxCatalog/SandboxCatalogBanner.tsx
Outdated
Show resolved
Hide resolved
workspaces/sandbox/plugins/sandbox/src/components/SandboxCatalog/SandboxCatalogBanner.tsx
Outdated
Show resolved
Hide resolved
workspaces/sandbox/plugins/sandbox/src/components/SandboxCatalog/SandboxCatalogBanner.tsx
Outdated
Show resolved
Hide resolved
…og/SandboxCatalogBanner.tsx Co-authored-by: Avik Kundu <avikkundu8d@gmail.com>
…og/SandboxCatalogBanner.tsx Co-authored-by: Avik Kundu <avikkundu8d@gmail.com>
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.
Thanks for the changes and the tests!
Looks like the tests are failing: https://github.com/redhat-developer/rhdh-plugins/actions/runs/14834610589/job/41643123310?pr=716 Can u check once? |
workspaces/sandbox/plugins/sandbox/src/components/SandboxCatalog/SandboxCatalogBanner.tsx
Outdated
Show resolved
Hide resolved
…og/SandboxCatalogBanner.tsx Co-authored-by: Avik Kundu <avikkundu8d@gmail.com>
|
Hey, I just made a Pull Request!
givenName
orcompliantUsername
endDate
provided by the backend , and not fixed to 30 days. Some users might have more than 30 days. Also the last day is excluded. The current logic was counting 31 days.endDate
is not setDepends on: codeready-toolchain/registration-service#529
Jiras: