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

Dev Feature access to Source view added line 151 #7

Closed
wants to merge 2 commits into from

Conversation

mikeadams1
Copy link

@mikeadams1 mikeadams1 commented Nov 3, 2019

Developer status will allow you to edit root index.html files. Navigate to https://sauser.solid.community/index.html and select Source code, click the pencil to edit, delete the existing file and insert new index.html file you want to appear on root of pod, click the green check mark. Your new homepage will be created on https://suaser.solid.community and your dashboard will be on https://sauser.solid.commuinty/index/html

Developer can edit root index.html files.
@brownhoward
Copy link

A user with any role, even the Normal User, can edit the root index.html file (based on permissions). We can include a note will instructions on how to edit the root index.html file using the Source view.

The Roles really affect only the panes/views that are visible for each resource.

@mikeadams1
Copy link
Author

To my knowledge, a normal user cannot edit index.html in source view without selecting developer status from preferences. Not to sure about power user though.

@megoth
Copy link

megoth commented Nov 7, 2019

A normal user don't have access to the Source view, which is what you're using to edit the HTML file. A normal user can still update the index.html through other means though (e.g. curl), so as the proposal currently stands it is easy to misinterpret.

If you believe that access to the Source view is an important part of the Developer user type, maybe you instead can change the existing point to include that as well?

Important for developers to have access to Source view and edit privileges.
@mikeadams1 mikeadams1 changed the title added line 151 Dev Feature access to Source view added line 151 Nov 7, 2019
@mikeadams1
Copy link
Author

If you believe that access to the Source view is an important part of the Developer user type, maybe you instead can change the existing point to include that as well?

Absolutely agree, in fact, more important for Developer user type to have access to Source view. That was a great feature to add @megoth.

A normal user can still update the index.html through other means though (e.g. curl), so as the proposal currently stands it is easy to misinterpret.

Yes, I agree, normal user can use curl to edit, among other means but, should not be able to do so via source view, I do consider that a feature.

@mikeadams1 mikeadams1 closed this Nov 7, 2019
@mikeadams1 mikeadams1 deleted the patch-1 branch November 7, 2019 11:42
@megoth
Copy link

megoth commented Nov 7, 2019

I'm sorry, I misremembered earlier - The Source view is in fact available for Normal users, so no need to change to Developer user type for that.

The feature to edit index.html through the data browser is a handy one, so we will take care to allow that in the future as well. But note that how it currently is available today is not necessarily the way we will make it available in the future (the current way is kind of a hack) - but if it changes we will notify the community in advance and update the Data Browser User Guide.

@mikeadams1
Copy link
Author

mikeadams1 commented Nov 7, 2019

The Source view is in fact available for Normal users, so no need to change to Developer user type for that.

on /index.html yes but user cannot edit it unless in developer mode, which is a feature if you ask me. Regular user should not be able to edit root /index.html unless maybe with curl or something else.

@megoth
Copy link

megoth commented Nov 7, 2019

The Source view is in fact available for Normal users, so no need to change to Developer user type for that.

on /index.html yes but user cannot edit it unless in developer mode, which is a feature if you ask me.

I do not understand what you mean by "unless in developer mode" - I just edited my https://megoth.solid.community/index.html by using the WebID that I use for that Pod (https://megoth.solid.community/profile/card#me), which is not assigned the Developer user type, i.e. it's a normal user. The way you phrase it sounds like you're saying it requires Developer mode now, but do you mean that it should require Developer mode?

Regular user should not be able to edit root /index.html unless maybe with curl or something else.

Why do you argue that regular users shouldn't be able to edit index.html via the data browser?

Do you argue that we should assign the Source view as a developer view, so people will have to assign themselves the Developer user type to be able to use it?

@mikeadams1
Copy link
Author

Ya know, I was dead wrong about that, I could have sworn that the other day I tried it on a new pod with a fellow dev and we could not get it to work unless in developer mode, my apologies.

Why do you argue that regular users shouldn't be able to edit index.html via the data browser?

I am not intentionally trying to argue the point that, a user should not be able to edit /index.html but, maybe a user should see a warning when scrolling over the Source view that indicates, any edits or changes the user may make, will effect users /index.html and dashboard. As a side note, if someone does unintentionally mess something up, at the present moment, as soon as you refresh the /index.html page and click on the Source view, click on the pencil, scroll to bottom, make a single edit of a space and then delete the space, publish by clicking the check mark, and the dashboard repopulates on root.

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.

None yet

3 participants