Skip to content

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

Merged
merged 10 commits into from
May 5, 2025

Conversation

mfrancisc
Copy link
Contributor

@mfrancisc mfrancisc commented May 2, 2025

Hey, I just made a Pull Request!

  • update welcome message in Sandbox plugin to use givenName or compliantUsername
  • the trial duration should be based on the 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.
  • avoid showing trial duration if endDate is not set

Depends on: codeready-toolchain/registration-service#529

Jiras:

@rhdh-gh-app
Copy link

rhdh-gh-app bot commented May 2, 2025

Missing Changesets

The following package(s) are changed by this PR but do not have a changeset:

  • @red-hat-developer-hub/backstage-plugin-sandbox

See CONTRIBUTING.md for more information about how to add changesets.

Changed Packages

Package Name Package Path Changeset Bump Current Version
@red-hat-developer-hub/backstage-plugin-sandbox workspaces/sandbox/plugins/sandbox none v0.0.1

@mfrancisc mfrancisc changed the title fix(sandbox): update welcome message fix(sandbox): update welcome message and fix trial duration May 2, 2025
Copy link
Contributor

@Lucifergene Lucifergene left a 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 :)

mfrancisc and others added 3 commits May 5, 2025 11:25
…og/SandboxCatalogBanner.tsx

Co-authored-by: Avik Kundu <avikkundu8d@gmail.com>
…og/SandboxCatalogBanner.tsx

Co-authored-by: Avik Kundu <avikkundu8d@gmail.com>
@mfrancisc mfrancisc requested a review from Lucifergene May 5, 2025 10:44
Copy link
Contributor

@Lucifergene Lucifergene left a 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!

@Lucifergene
Copy link
Contributor

Looks like the tests are failing: https://github.com/redhat-developer/rhdh-plugins/actions/runs/14834610589/job/41643123310?pr=716

Can u check once?

mfrancisc and others added 2 commits May 5, 2025 16:37
…og/SandboxCatalogBanner.tsx

Co-authored-by: Avik Kundu <avikkundu8d@gmail.com>
Copy link

sonarqubecloud bot commented May 5, 2025

@Lucifergene Lucifergene merged commit 3f39532 into redhat-developer:main May 5, 2025
9 checks passed
@mfrancisc mfrancisc deleted the fixfewthings branch May 5, 2025 17:37
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