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

Feature: Protected articles #343

Open
imarcu opened this issue Nov 6, 2018 · 1 comment
Open

Feature: Protected articles #343

imarcu opened this issue Nov 6, 2018 · 1 comment
Labels
enhancement New feature or request help wanted Extra attention is needed question Further information is requested

Comments

@imarcu
Copy link

imarcu commented Nov 6, 2018

Expanding #338 to a more flexible feature.

bool IsProtected { get; set; }

This property should not allow for the following:

  • renames
  • deletions
  • content changes.

By default this value should be false for all articles except Home page.
could be modified only by an admin for each article.

Possible issues when merging #338 :

  • home page needs to be differentiated for UI changes (hide back button)
@csharpfritz csharpfritz added enhancement New feature or request help wanted Extra attention is needed question Further information is requested labels Nov 6, 2018
@tiagodenoronha
Copy link

In my understanding, this shouldn't be applied to the HomePage.
Think of wikipedia when you have articles only some people can edit them. This applies perfectly to that case.

The problem is HomePage, FAQ, News aren't an article.
Maybe the HomePage has various images, FAQ has a list of questions, etc etc.

I believe this should be a different class / table.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants