Skip to content

add-authorization#9

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

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

Conversation

@jsonh0

@jsonh0 jsonh0 commented Oct 23, 2023

Copy link
Copy Markdown

No description provided.

@Samuel-Lubliner Samuel-Lubliner left a comment

Copy link
Copy Markdown

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

@thatcantbeso thatcantbeso left a comment

Copy link
Copy Markdown

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.

@hibbleD hibbleD left a comment

Copy link
Copy Markdown

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