vaadin / hilla

Build better business applications, faster. No more juggling REST endpoints or deciphering GraphQL queries. Hilla seamlessly connects Spring Boot and React to accelerate application development.
https://hilla.dev
Apache License 2.0
904 stars 57 forks source link

fix: nested cases of route parameters in the menu #2527

Closed platosha closed 3 months ago

platosha commented 3 months ago

Fixes #2378

codecov[bot] commented 3 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Please upload report for BASE (24.4@c044d28). Learn more about missing BASE report.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## 24.4 #2527 +/- ## ======================================= Coverage ? 95.01% ======================================= Files ? 66 Lines ? 4557 Branches ? 665 ======================================= Hits ? 4330 Misses ? 182 Partials ? 45 ``` | [Flag](https://app.codecov.io/gh/vaadin/hilla/pull/2527/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=vaadin) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/vaadin/hilla/pull/2527/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=vaadin) | `95.01% <100.00%> (?)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=vaadin#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

sonarcloud[bot] commented 3 months ago

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

vaadin-bot commented 3 months ago

This ticket/PR has been released with Hilla 24.4.0.rc2 and is also targeting the upcoming stable 24.4.0 version.

vaadin-bot commented 3 months ago

Hi @platosha and @platosha, when i performed cherry-pick to this commit to main, i have encountered the following issue. Can you take a look and pick it manually? Error Message: Error: Command failed: git cherry-pick 944678fee13dd13ae0816809c0c06b25393e33e5 error: could not apply 944678fe... fix: nested cases of route parameters in the menu (#2527) hint: After resolving the conflicts, mark them with hint: "git add/rm ", then run hint: "git cherry-pick --continue". hint: You can instead skip this commit with "git cherry-pick --skip". hint: To abort and get back to the state before "git cherry-pick", hint: run "git cherry-pick --abort".