Skip to content

Update permission.js#206

Open
dioptre wants to merge 1 commit into
trailsjs:masterfrom
dioptre:patch-1
Open

Update permission.js#206
dioptre wants to merge 1 commit into
trailsjs:masterfrom
dioptre:patch-1

Conversation

@dioptre

@dioptre dioptre commented Jan 8, 2016

Copy link
Copy Markdown

Setting default read for everything and making it hard to change is bad, especially after loading large user sets.

Setting default read for everything and making it hard to change is bad, especially after loading large user sets.
@ryanwilliamquinn

Copy link
Copy Markdown
Contributor

I think an option where you can pass in a config might be a better solution. So you can specify in config/permissions.js that you want registered to have no permissions by default.

module.exports.permissions = {
    grant: {
        registered: []
    }
}

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.

2 participants