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.

Sign up for GitHub

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

Jump to bottom

Check authorization #8

Closed
wants to merge 2 commits into from
Closed

Check authorization #8

wants to merge 2 commits into from

Conversation

zzuutt
Copy link
Contributor

@zzuutt zzuutt commented May 12, 2016

Add authorization control
if the user is authorized to view this module

@zzuutt
Check authorization
cc307dc 
Add  authorization control
if the user is authorized to view this module
@Soldras
Copy link
Member

Soldras commented May 12, 2016

Hi,
Squash your commit please.

@zzuutt
Copy link
Contributor Author

zzuutt commented May 13, 2016

git respond
$ git rebase -i master
Successfully rebased and updated refs/heads/patch-1.
$ git push -f origin patch-1
Everything up-to-date

but nothing is done

@Soldras
Copy link
Member

Soldras commented May 13, 2016

Hum.. Weird.

Check this doc.

@zzuutt zzuutt closed this Jun 2, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
bug enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@zzuutt @Soldras