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

Michael's Instagram Challenge #917

Open
wants to merge 19 commits into
base: main
Choose a base branch
from
Open

Michael's Instagram Challenge #917

wants to merge 19 commits into from

Conversation

mcsuGH
Copy link

@mcsuGH mcsuGH commented May 15, 2022

Implemented most of the functionality (can make post, comment on posts, like on posts). Users can sign in and sign out and conditionals are used to show content depending on whether or not the user is signed in or not (for example, if you are signed in, the Sign In button at the navigation bar becomes a Log Out, or if you are already logged in and you go to the Login Page it will tell you you are already logged in.)

Will need to add functionality to upload images to database (currently only allows URL of external image hosts like imgur) and still need to add more CSS to make it look nicer.

Copy link

@JordanManu JordanManu left a comment

Choose a reason for hiding this comment

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

Great project! It works really well, I love the comment feature that displays who commented and the time it was posted. Edge cases were covered and great test coverage!

- add more CSS - currently only added a bar at the top and a background for the post images, displaying the images in rows of 3.
- make it so each user can only like a post once (would just need to create a new collection for likes inside database with foreign keys for the post and the user who liked the post)
- add filter function
- add the ability to upload images onto the database rather than using a external image host

Choose a reason for hiding this comment

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

Great readme instructions, very easy to follow

})
}
}

Choose a reason for hiding this comment

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

all v nice!

cy.get(".createpost").click();

cy.url().should("include", "/posts");
cy.get('div[class="posts"]').find(".postimage").should('be.visible');

Choose a reason for hiding this comment

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

I like this test!


cy.url().should("include", "/sessions");
cy.should("not.contain", "Hey someone3@example.com");
});

Choose a reason for hiding this comment

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

Good test

cy.get("#password").type("password");
cy.get("#submit").click();

cy.contains("Email already in use");

Choose a reason for hiding this comment

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

nice!

expect(err).toBeNull();

expect(posts[0].createdAt).toBeDefined();
done();

Choose a reason for hiding this comment

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

Great tests!

{{/if}}
</div>
{{#if user}}
Hey {{user.email}}

Choose a reason for hiding this comment

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

Really like this feature

<input type="text" id="comment" name="comment" placeholder="write comment" required>
</div>
<div>
<button class ='createcomment'>Comment</button>

Choose a reason for hiding this comment

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

Real good

</div>
<br>
<form action = '/posts'>
<button class = "back">back</button>

Choose a reason for hiding this comment

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

I like the back button

@@ -0,0 +1,17 @@
{{#if user}}
You are already logged in.

Choose a reason for hiding this comment

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

Great

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.

2 participants