loopbackio / loopback-next

LoopBack makes it easy to build modern API applications that require complex integrations.
https://loopback.io
Other
4.95k stars 1.07k forks source link

Update controller class in package `rest-crud` when id is a required property #3728

Closed jannyHou closed 3 years ago

jannyHou commented 5 years ago

connect to https://github.com/strongloop/loopback-next/issues/2036 This is a follow-up story of PR https://github.com/strongloop/loopback-next/pull/3667. As a quick fix, that PR fixes the controller generator by adding a new prompt isIdOmitted, and creating the corresponding controller class differently.

There are two more improvements which need more time to address, so I open a new issue here.

Suggestion

Update the controller class in @loopback/rest-crud to reflect the id property's configuration.

Acceptance criteria

bajtos commented 5 years ago

(Not sure is it a good idea to do them in one story, or should I create two?)

Let's create two stories please.

stale[bot] commented 4 years ago

This issue has been marked stale because it has not seen activity within six months. If you believe this to be in error, please contact one of the code owners, listed in the CODEOWNERS file at the top-level of this repository. This issue will be closed within 30 days of being stale.

stale[bot] commented 3 years ago

This issue has been closed due to continued inactivity. Thank you for your understanding. If you believe this to be in error, please contact one of the code owners, listed in the CODEOWNERS file at the top-level of this repository.