c2corg / v6_ui

UI for c2c.org v6
GNU Affero General Public License v3.0
7 stars 12 forks source link

Handle protected documents #239

Closed tsauerwein closed 7 years ago

tsauerwein commented 8 years ago

API issue: https://github.com/c2corg/v6_api/issues/217

With https://github.com/c2corg/v6_api/pull/191 the protected field of documents will be included. Protected documents can only be edited by moderators.

In the UI maybe we can disable the edit button and show a hover message ("Document is protected, ...") when a user is not moderator. The edit page should also show a "not authorized" message and the form should not be shown.

Example for a protected document in v5: http://www.camptocamp.org/summits/213785/it/piramide-ovest-di-somalbosco

asaunier commented 8 years ago

The protection status of a document is set in a document attribute, right? Wich means we could have a "protected" attribute in the editing forms only available to the moderators?

tsauerwein commented 8 years ago

I was thinking about providing a separate web-service for changing the protected status (similar to merging a document). The protected checkbox might be a bit lost in the normal editing form, so maybe it's better to have an explicit command to lock/unlock a document (as part of the moderator tools).

asaunier commented 8 years ago

In v5, the lock/unlock function is indeed a separated function.

Wouldn't it save time to have a checkbox in the editing forms (only for moderators)? We would not have to create a dedicated webservice, it's quite similar to the "quality" attribute (~metadata), and locking/unlocking creates a new version in the history anyway (doesn't it?).

Since it is a special tool for advanced users, it's not that a big deal if it is a bit lost. As I said it could be in a metadata section placed near the "save" button, containing also the quality attribute and the "comments about the change" attribute.

tsauerwein commented 8 years ago

Wouldn't it save time to have a checkbox in the editing forms (only for moderators)? We would not have to create a dedicated webservice, it's quite similar to the "quality" attribute (~metadata), and locking/unlocking creates a new version in the history anyway (doesn't it?).

For me it would actually be easier to have a separate web-service.

asaunier commented 8 years ago

OK then. I have created an issue about that in the API: https://github.com/c2corg/v6_api/issues/217

I guess it would be a function available in the "More" menu button.

asaunier commented 8 years ago

@fjacon This task cannot be done until the API webservice for it is available: https://github.com/c2corg/v6_api/issues/217

desnoes commented 7 years ago

It works correctly on demo