Skip to content

Conversation

@odeimaiz
Copy link
Member

@odeimaiz odeimaiz commented Oct 7, 2025

What do these changes do?

reported by @nereaiscamu

This PR fixes the "My Account" functionality for the s4llite product by updating the switch case logic in the product utilities. The change ensures that s4llite users are properly routed to the university-specific account configuration instead of falling through to the default case.

Related issue/s

How to test

Dev-ops

@odeimaiz odeimaiz self-assigned this Oct 7, 2025
@odeimaiz odeimaiz added this to the Cheops milestone Oct 7, 2025
@odeimaiz odeimaiz added bug buggy, it does not work as expected a:frontend issue affecting the front-end (area group) labels Oct 7, 2025
@sonarqubecloud
Copy link

sonarqubecloud bot commented Oct 7, 2025

@odeimaiz odeimaiz requested a review from Copilot October 7, 2025 09:36
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 fixes the "My Account" functionality for the s4llite product by updating the switch case logic in the product utilities. The change ensures that s4llite users are properly routed to the university-specific account configuration instead of falling through to the default case.

  • Replaced "s4ldesktopacad" case with "s4llite" case to match the correct product identifier
  • Added explicit default case to improve code clarity and prevent unintended fallthrough behavior

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@mergify
Copy link
Contributor

mergify bot commented Oct 7, 2025

🧪 CI Insights

Here's what we observed from your CI run for 6298647.

🟢 All jobs passed!

But CI Insights is watching 👀

@odeimaiz odeimaiz requested a review from nereaiscamu October 7, 2025 09:56
@odeimaiz odeimaiz merged commit f811fc3 into ITISFoundation:master Oct 7, 2025
58 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a:frontend issue affecting the front-end (area group) bug buggy, it does not work as expected

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants