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

Business menu: possibility to mask parameters when activating a specific menu iFrame #5013

Closed
OlivierVoron opened this issue Sep 13, 2023 · 3 comments · Fixed by #5050
Closed
Assignees

Comments

@OlivierVoron
Copy link

Actually, the function templateGateway.redirectToBusinessMenu gives the possibility to redirect to a specific business menu from inside a card.
The actual implementation adds the additional parameters in the main url of OperatorFabric shown in the web browser and the end-user may have the possibility to record this url as a bookmark or see the parameters provided.
As expected and wanted, these additional parameters are also provided to the iFrame called inside OperatorFabric.

Unfortunately, if the user navigates in the application inside the iFrame, the main url of OperatorFabric will not change and it is possible that the content of this url with parameters mismatch with the real content shown by the business application in the iFrame.

Expected behaviour:
The proposed correction is to mask the parameters in the main url: the redirect uri will always be the same as the one that the users could select as a menu/sub-menus in the navigation bar.

@freddidierRTE
Copy link
Contributor

i found a solution to do it using history.replaceState js function but i wanted to know if this new behavior is acceptable for everybody before implementing it
If not , I'm open to making it configurable so that it can be adjusted to meet individual needs.

@freddidierRTE freddidierRTE self-assigned this Sep 18, 2023
@OlivierVoron
Copy link
Author

For us, the solution seems fine

@geppyz
Copy link
Contributor

geppyz commented Sep 28, 2023

I would also like to report it works fine in our setup. No problems found during testing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants