Implemented spring implementation. #4
Conversation
| @@ -0,0 +1,106 @@ | |||
| # DemoApplication | |||
|
|
|||
| A webforJ application powered by Spring Boot. This project combines the power of webforJ framework with Spring Boot's comprehensive ecosystem for building enterprise applications. | |||
There was a problem hiding this comment.
This project combines webforJ's front end framework with Spring Boot's...
|
|
||
| ### Spring Boot Features | ||
|
|
||
| This project leverages Spring Boot's features: |
There was a problem hiding this comment.
This project demonstrates the following Spring Boot features:
| Your application will automatically restart when files on the classpath change. | ||
|
|
||
|
|
||
| ## Running Integration Tests |
There was a problem hiding this comment.
Are you planning to include tests for each step? If not, remove this section.
| @@ -0,0 +1,55 @@ | |||
| package com.demoapplication.views; | |||
There was a problem hiding this comment.
Either update the test to reflect current app, or remove.
spring/2-working-with-data/README.md
Outdated
|
|
||
| ### Spring Boot Features | ||
|
|
||
| This project leverages Spring Boot's features: |
There was a problem hiding this comment.
Change to the same as the correction in step 1's README
|
|
||
| @Repository | ||
| public interface CustomerRepository extends JpaRepository<Customer, Long>, JpaSpecificationExecutor<Customer> { | ||
|
|
There was a problem hiding this comment.
Hyyan is going to love this empty space ;)
| @@ -0,0 +1,55 @@ | |||
| package com.demoapplication.views; | |||
There was a problem hiding this comment.
Remove if not being updated
| public DemoView(CustomerService customerService) { | ||
| this.customerService = customerService; | ||
|
|
||
| add.setStyle("margin", "var(--dwc-space-l)").setWidth(200); |
There was a problem hiding this comment.
I'd add a classname and move this styling to the CSS file
| } | ||
|
|
||
| @Override | ||
| public void onDidEnter(DidEnterEvent event, ParametersBag parameters) { |
There was a problem hiding this comment.
I forget why we use the observer here instead of putting this in the constructor? If that is in fact necessary, let's make sure to specify why in the documentation page created for this section.
| TextField lastName = new TextField("Last Name"); | ||
| TextField company = new TextField("Company"); | ||
| ChoiceBox country = new ChoiceBox("Country"); | ||
| TextField firstName = new TextField("First Name", e -> customer.setFirstName(e.getValue())); |
There was a problem hiding this comment.
Shouldn't need to manually set the info if you're using data binding
No description provided.