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

Add a UI to add supported permissions to an application #44

Merged
merged 4 commits into from Jan 15, 2013

Conversation

davidillsley
Copy link
Contributor

Minimum changes to allow us to add a new permission when we add an authority to licensing. Which means no editing/deletion of permissions.

@mnowster
Copy link
Contributor

I think this reminds us that we need to have a conversation about a possible permission naming convention and needing finer grained permissions ourselves in sign on so that this functionality isn't open to everyone.

end
end

context "GET create" do
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be "POST create"

*   GET create -> POST create
*   Only do a single request in the test
mnowster added a commit that referenced this pull request Jan 15, 2013
Add a UI to add supported permissions to an application
@mnowster mnowster merged commit b05f5c4 into master Jan 15, 2013
@benilovj benilovj deleted the add-supported-permissions-ui branch December 5, 2013 11:42
gpeng pushed a commit that referenced this pull request Nov 29, 2017
Fix creation of dummy API user in test mode
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