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

Rename acl:defaultForNew to acl:default #100

Open
1 of 4 tasks
dmitrizagidulin opened this issue Apr 6, 2016 · 0 comments
Open
1 of 4 tasks

Rename acl:defaultForNew to acl:default #100

dmitrizagidulin opened this issue Apr 6, 2016 · 0 comments

Comments

@dmitrizagidulin
Copy link
Member

dmitrizagidulin commented Apr 6, 2016

As discussed during several dev meetings, the semantics of the currently implemented acl:defaultForNew predicate are slightly off, and are more appropriate to be named acl:default.

The consensus was to rename existing usage of acl:defaultForNew predicate in the ACLs of Gold and LDNode installations. (And also have a transitionary period where the server supports both predicates for a few versions, and just gives a deprecation warning for defaultForNew).

To do:

  • implement support for acl:default in solid-permissions (it should also handle legacy acl:defaultForNew)
  • implement support for acl:default in solid-ui/src/acl.js
  • implement support for acl:default in node-solid-server (switch default account template to use it)
  • mass-rename existing ACL files (using sed or equivalent) on solid.community and solidtest.space
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

No branches or pull requests

1 participant