Skip to content

Conversation

big-cir
Copy link
Contributor

@big-cir big-cir commented Apr 22, 2025

Overview

  • Updated WebMvcProperties to include a nested Jsp class for hierarchical binding.
  • Registered a jspViewResolver bean when spring.mvc.jsp.prefix and spring.mvc.jsp.suffix are set.
  • Added tests to verify property binding for JSP-specific prefix and suffix.

Related

ISSUE: #44924

Note

I’m aware that the issue is currently labeled with status: pending-design-work, which suggests that the final design direction might still be under consideration. This implementation reflects my current understanding of the issue and is intended to serve as a starting point for discussion.

If this approach doesn't align with the intended design direction, please feel free to close the PR or share feedback — I’m happy to revise or take a different direction as needed. Thank you!

@big-cir big-cir force-pushed the refactor/mvc-jsp-properties branch from 98eaee3 to 0042281 Compare April 22, 2025 14:52
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Apr 22, 2025
@philwebb
Copy link
Member

Thanks for your efforts here @big-cir, but issue #44924 is labeled as pending-design-work which means we aren't ready to accept contributions for it.

@philwebb philwebb closed this Apr 22, 2025
@philwebb philwebb added status: declined A suggestion or change that we don't feel we should currently apply and removed status: waiting-for-triage An issue we've not yet triaged labels Apr 22, 2025
@big-cir
Copy link
Contributor Author

big-cir commented Apr 22, 2025

Thanks for your efforts here @big-cir, but issue #44924 is labeled as pending-design-work which means we aren't ready to accept contributions for it.

@philwebb
Thank you for letting me know! I'll keep an eye on the issue and check back once the design work is complete. Have a great day!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: declined A suggestion or change that we don't feel we should currently apply

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants