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

Make all navigation bar links visible to all users #803

Merged
merged 2 commits into from
Oct 21, 2024

Conversation

ashanhr
Copy link
Contributor

@ashanhr ashanhr commented Oct 14, 2024

Purpose

Make all navigation bar links visible to all users, as anyone who can log in to the publisher portal can view every page.
Fix FormattedMessage usage for correct string passing.
Related Issue: wso2/api-manager#3118

Approach

Remove user role checks from navigation bar.
Replaced the direct usage of FormattedMessage with intl.formatMessage.

Copy link

@RakhithaRR RakhithaRR merged commit 43237e1 into wso2:main Oct 21, 2024
3 checks passed
hisanhunais pushed a commit to hisanhunais/apim-apps that referenced this pull request Jan 23, 2025
hisanhunais pushed a commit to hisanhunais/apim-apps that referenced this pull request Jan 23, 2025
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