Skip to content
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

fix(frontend): Add error page for unsubscribed users #87

Merged

Conversation

fabiodmota
Copy link
Contributor

@fabiodmota fabiodmota commented Feb 16, 2024

Description

Added Error Page for when user is with right credentials
Users now cant access Country Risk if the subscription is not found on the token information .
User can be auth but the sub may not exist
Users with auth working on Portal and subscribed to Country Risk can access

#74

Pre-review checks

Please ensure to do as many of the following checks as possible, before asking for committer review:

Copy link
Contributor

@alexsilva-CGI alexsilva-CGI left a comment

Choose a reason for hiding this comment

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

Looks good

@fabiodmota fabiodmota requested a review from jjeroch February 19, 2024 10:00
@fabiodmota
Copy link
Contributor Author

image

Error page implemented from Portal Components https://eclipse-tractusx.github.io/portal-shared-components/?path=/docs/errorpage--docs

@fabiodmota fabiodmota force-pushed the fix/page_redirect_for_portal branch from 87149d2 to dcf8533 Compare February 19, 2024 13:59
@fabiodmota fabiodmota requested a review from evegufy February 19, 2024 13:59
@evegufy evegufy changed the title Fix(Frontend): Added error page for unsubscribed users fix(frontend): Add error page for unsubscribed users Feb 19, 2024
@evegufy evegufy merged commit 86fb5b9 into eclipse-tractusx:main Feb 19, 2024
3 checks passed
@fabiodmota fabiodmota deleted the fix/page_redirect_for_portal branch February 20, 2024 09:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

4 participants