-
Notifications
You must be signed in to change notification settings - Fork 10
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
Published flag for documents #291
Comments
I find the semantic distinction between published versus viewable-by-all to be rather subtle. Presumably any published document must be at least viewable-by-all, so these can't quite be independent columns. Should we just force permissions up to at least "all viewable" if the user checks published? |
I agree that it's subtle. I think the distinction is that a published document broadcasts its existence (e.g., it shows up in public lists of a user's models or in searches of the public library of models), whereas as a non-published document, even if readable by anyone, can only be found if you have the link. Here's what the n-dim report (Sec 3.1.2) says about publishing:
We will have to confront the same sorts of issues one way or another once models can depend on other models, if we want to be a proper "GitHub for models." GitHub itself doesn't quite have this problem because repos cannot depend on other repos (except through Git submodules, which basically no one uses or thinks about). |
Having just been talking about snapshots over at #318 , I wonder whether it's really a snapshot that should be published, rather than a document? Of course we still aren't saving snapshots except at the head... |
In addition to its permissions/access control list, a document should have a
published
flag stating whether it is publicly listed. Once a document is published, it cannot be unpublished, and a published document can depend only on other published documents.This requires:
published
to the documentref
tableFor further discussion, see Sec 3.1.2 of the n-dim report.
The text was updated successfully, but these errors were encountered: