Skip to content

Conversation

@odeimaiz
Copy link
Member

@odeimaiz odeimaiz commented Jun 11, 2025

What do these changes do?

If the product is configured to request an account via the form, the "Request account" flash message shown to the guest users will show a link to the request-account form instead of the send-us-an-email approach.

When request account goes through the form:
RequestAccount1

When the product config expects an email:
RequestAccount

Related issue/s

How to test

Dev-ops

@odeimaiz odeimaiz self-assigned this Jun 11, 2025
@odeimaiz odeimaiz changed the title refactor [Frontend] Redirect anonymous users to the "account request form" instead of asking them to send an email Jun 11, 2025
@odeimaiz odeimaiz changed the title [Frontend] Redirect anonymous users to the "account request form" instead of asking them to send an email 🎨 [Frontend] Redirect anonymous users to the "account request form" instead of asking them to send an email Jun 11, 2025
@odeimaiz odeimaiz requested a review from Copilot June 11, 2025 15:16
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the account request flash message for guest users so that, when configured, a link to the request-account form is displayed instead of prompting the user to send an e-mail.

  • Updated flash message generation in osparc/utils/Utils.js to branch between a request form link and a mailto link.
  • Introduced a new method (requestAccountLink) in osparc/store/Support.js to generate the request-account link with optional centering.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
services/static-webserver/client/source/class/osparc/utils/Utils.js Adjusts message concatenation and adds a branch for the REQUEST_ACCOUNT_FORM action
services/static-webserver/client/source/class/osparc/store/Support.js Adds the new requestAccountLink() method to build the request-account link

@odeimaiz odeimaiz added t:enhancement Improvement or request on an existing feature a:frontend issue affecting the front-end (area group) 🤖-automerge marks PR as ready to be merged for Mergify labels Jun 11, 2025
@odeimaiz odeimaiz added this to the Engage milestone Jun 11, 2025
@odeimaiz
Copy link
Member Author

@Mergifyio queue

@odeimaiz odeimaiz marked this pull request as ready for review June 11, 2025 15:19
@mergify
Copy link
Contributor

mergify bot commented Jun 11, 2025

queue

✅ The pull request has been merged automatically

The pull request has been merged automatically at 73a45bc

Copy link
Member

@pcrespov pcrespov left a comment

Choose a reason for hiding this comment

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

thx so much!

@sonarqubecloud
Copy link

@mergify mergify bot merged commit 73a45bc into ITISFoundation:master Jun 11, 2025
59 checks passed
@matusdrobuliak66 matusdrobuliak66 mentioned this pull request Aug 5, 2025
88 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🤖-automerge marks PR as ready to be merged for Mergify a:frontend issue affecting the front-end (area group) t:enhancement Improvement or request on an existing feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Redirect anonymous users to the "account request form" isntead of asking them to send an email

4 participants