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

Branch c #2

Merged
merged 11 commits into from
Jul 26, 2020
Merged

Branch c #2

merged 11 commits into from
Jul 26, 2020

Conversation

Laguna1
Copy link
Owner

@Laguna1 Laguna1 commented Jul 23, 2020

We worked with this project to study active records, models and associations
We built a project similar to Reddit, in which we created several tables - users, posts, and comments to posts
to save all information about the user and his publications.
Users, posts, and comments are linked by relevant associations.
A user can have many posts and many comments, but a post can have only one user and can be posted in one post.
Thanks for your comments.

Copy link

@wathigo wathigo left a comment

Choose a reason for hiding this comment

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

Changes Required.

Great job on this project.
I could not approve of this project because of one issue that needs to be addressed.

REQUIRED

Make the necessary changes and submit another request for review.

All the best 🍀
Simon Wathigo.

@Laguna1
Copy link
Owner Author

Laguna1 commented Jul 23, 2020

Thanks for your comments.
We made the necessary changes in our Readme.file.

Copy link

@rOluochKe rOluochKe left a comment

Choose a reason for hiding this comment

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

Hello Team!

Well done, you got this right 👏 The project has been approved. Congratulations 💥 💯

Merge and submit the project. Good luck with the next one.

  • Happy coding!

@Laguna1 Laguna1 merged commit ea9cfc8 into development Jul 26, 2020
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