Skip to content
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

Update crud.md #16791

Closed
wants to merge 2 commits into from
Closed

Update crud.md #16791

wants to merge 2 commits into from

Conversation

Rick-Anderson
Copy link
Contributor

@Rick-Anderson Rick-Anderson commented Jan 31, 2020

This is preparing the doc for when @wadepickett does the 5.0 preview update he can also do #13752

@Rick-Anderson Rick-Anderson marked this pull request as ready for review January 31, 2020 02:49
### View model

View models provide an alternative way to prevent overposting.
### Input model and view model
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I recommend sticking with the original heading. "View model" is an industry term.


The application model is often called the domain model. The domain model typically contains all the properties required by the corresponding entity in the database. The view model contains only the properties needed for the UI that it is used for (for example, the Create page).
An input model or view model provides an alternative way to prevent overposting.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same suggestion here. I wouldn't recommend using the term "input model".

@Rick-Anderson Rick-Anderson deleted the Rick-Anderson-patch-1 branch January 31, 2020 17:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants