Skip to content

Fully complete I hope#3

Open
armstrca wants to merge 4 commits intoappdev-projects:mainfrom
armstrca:main
Open

Fully complete I hope#3
armstrca wants to merge 4 commits intoappdev-projects:mainfrom
armstrca:main

Conversation

@armstrca
Copy link
Copy Markdown
Collaborator

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.

Cant add post. Might want to try calling authorize in PhotosController and updating PhotoPolicy.

Copy link
Copy Markdown

@johnLeavell johnLeavell left a comment

Choose a reason for hiding this comment

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

Unable to add a photo.

something to note: In production commented code is okay but before pushing to production you want to remove any uncommented / unused code.

def photo_params
params.require(:photo).permit(:image, :comments_count, :likes_count, :caption, :owner_id)
end

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

having comments in development but when pushing to production you want to remove any unused / commented code

@Dantexkilljoy
Copy link
Copy Markdown

This is cool but its going to app dev. Other than that, you chillin ~with-rizz

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.

4 participants