-
Notifications
You must be signed in to change notification settings - Fork 2
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
LPD-32573 Use the url mapping to avoid missing parameters on language change #5862
LPD-32573 Use the url mapping to avoid missing parameters on language change #5862
Conversation
… to make it clearer and safer.
CI is automatically triggering the following test suites:
|
✔️ ci:test:sf - 1 out of 1 jobs passed in 4 minutesClick here for more details.Base Branch:Branch Name: master Sender Branch:Branch Name: LPD-32573 1 Successful Jobs:For more details click here. |
Jenkins Build:test-portal-source-format#8046 Jenkins Report:jenkins-report.html Jenkins Suite:sf Pull Request:liferay-content-management#5862 Testray Routine:EE Pull Request Testray Build:[master] ci:test:sf - robertoDiaz > liferay-content-management - PR#5862 - 2024-09-12[03:57:23] Testray Build ID:9694787 Testray Importer:publish-testray-report#4420 |
ci:test:relevant |
ci:test:content-management |
Jenkins Build:test-portal-acceptance-pullrequest(master)#6606 Jenkins Report:jenkins-report.html Jenkins Suite:relevant Pull Request:liferay-content-management#5862 Testray Routine:EE Pull Request Testray Build:[master] ci:test:relevant - robertoDiaz > liferay-content-management - PR#5862 - 2024-09-12[05:33:27] Testray Build ID:10120153 Testray Importer:publish-testray-report#14523 |
Jenkins Build:test-portal-acceptance-pullrequest(master)#17906 Jenkins Report:jenkins-report.html Jenkins Suite:content-management Pull Request:liferay-content-management#5862 Testray Routine:EE Pull Request Testray Build:[master] ci:test:content-management - robertoDiaz > liferay-content-management - PR#5862 - 2024-09-12[05:27:43] Testray Build ID:10210294 Testray Importer:publish-testray-report#38755 |
ci:forward |
ci:forward |
CI is automatically triggering the following test suites:
The pull request will automatically be forwarded to the user
|
Skipping previously passed test suites:
|
Jenkins Build:test-portal-acceptance-pullrequest(master)#11025 Jenkins Report:jenkins-report.html Jenkins Suite:relevant Pull Request:liferay-content-management#5862 Testray Routine:EE Pull Request Testray Build:[master] ci:test:relevant - robertoDiaz > liferay-content-management - PR#5862 - 2024-09-17[08:16:13] Testray Build ID:13709689 Testray Importer:publish-testray-report#23761 |
ci:report:13709689 |
Build completed.Testray CSV has been generated successfully for testrayBuildID: 13709689. Job Link: generate-testray-csv Testray CSV Link: testray-results-13709689.csv |
@adolfopa Test seem unrelated. |
ci:forward:force |
CI is automatically triggering the following test suites:
The pull request will automatically be forwarded to the user
|
Skipping previously completed test suites:
|
All required test suite(s) completed. |
Pull request has been successfully forwarded to brianchandotcom#154328 |
What is this trying to solve?
https://liferay.atlassian.net/browse/LPD-32573
How am I fixing it?
Using the existing implementations of FriendlyURLMapper to get the full url and avoid missing parameters
How can you verify that it works?
Run the included automated tests.