Skip to content
This repository has been archived by the owner on Jun 2, 2023. It is now read-only.

spring mvc stuff #171

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

spring mvc stuff #171

wants to merge 1 commit into from

Conversation

paulben
Copy link
Contributor

@paulben paulben commented Apr 30, 2017

The springboot starter appears to me to be Spring REST but not Spring MVC as it uses @RestController, which is not the same as a @controller which is generally used for MVC.

This PR contains the changes I made to add an MVC @controller. It's not complete and probably needs design work (probably Spring MVC should be a separate technology from Spring REST?) but I just wanted to save the changes I made - I didn't update test and no doubt didn't dot all the i's and cross all the t's...etc., etc.

It basically:

  • adds a thymeleaf html (hello.html) which uses data from the model
  • adds a @controller (LibertyMVController.java) which updates the model with a request param
  • renames LibertyHelloController.java to LibertyRESTController.java
  • updates the compile-pom to add thymeleaf
  • updates description.html to reference the new REST and MVC endpoints.

This change is Reviewable

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant