Skip to content

add-authorization#9

Open
jsonh0 wants to merge 5 commits intoappdev-projects:mainfrom
jsonh0:main
Open

add-authorization#9
jsonh0 wants to merge 5 commits intoappdev-projects:mainfrom
jsonh0:main

Conversation

@jsonh0
Copy link
Copy Markdown

@jsonh0 jsonh0 commented Oct 23, 2023

No description provided.

Copy link
Copy Markdown

@Samuel-Lubliner Samuel-Lubliner left a comment

Choose a reason for hiding this comment

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

give user permission to add post. Try to call authorize in create photo and methods in PhotoPolicy

Copy link
Copy Markdown

@thatcantbeso thatcantbeso left a comment

Choose a reason for hiding this comment

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

I think you need some work on this :|

Comment thread app/controllers/application_controller.rb
Comment thread app/controllers/comments_controller.rb Outdated
Comment thread app/controllers/likes_controller.rb Outdated
Comment thread app/controllers/photos_controller.rb Outdated
Comment thread app/controllers/users_controller.rb Outdated
@Dantexkilljoy
Copy link
Copy Markdown

👍 just make sure you make a branch next time for this.

Copy link
Copy Markdown

@hibbleD hibbleD left a comment

Choose a reason for hiding this comment

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

Looks good!

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.

5 participants