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 2022-01-11-orm-2.11.md to be clear about some limitations over "Support for Readonly Properties" #479

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

Conversation

n4huel
Copy link

@n4huel n4huel commented Apr 17, 2023

@SenseException
Copy link
Member

Hi @n4huel, and thank you for your contribution. This is a blog post from January 2022 about ORM 2.11. I don't know if an old blog post is supposed to be changed.

@beberlei how do you see it? should we adapt an old blog post? I'm asking you because this is one you've created.

@n4huel
Copy link
Author

n4huel commented Apr 19, 2023

Thanks you @SenseException, your point is accurate. Maybe it should include some note/disclaimer update as "errata" since the feature isn't really supported.

@stof
Copy link
Member

stof commented Aug 23, 2023

"it will not work" is also limited to the id field. Other fields can still be readonly.

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.

3 participants