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.

Sign up for GitHub

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

Jump to bottom

Chitter app 0.1 #1605

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

Chitter app 0.1 #1605

wants to merge 48 commits into from

Conversation

Iliyan-Y
Copy link

@Iliyan-Y Iliyan-Y commented Aug 30, 2020 •
edited
Loading

Features:

  • all users can see all peeps and order them
  • user can create account
  • Registred users can post new peeps
  • use can log in / out
  • User name displayed next to each post

NOT implemented yet:

  • User can post comments
  • User receive email when is tagged in comment

Iliyan-Y added 30 commits August 29, 2020 14:41
@Iliyan-Y
Envroiment setup
65247e8
@Iliyan-Y
Database setup added
9521e71
@Iliyan-Y
Database connection setup added
8e1e1d8
@Iliyan-Y
Methods for getting and creating posts added to the monedel
c64b85c
@Iliyan-Y
User can post and see all previouse posts
ebceea3
@Iliyan-Y
User can choose the display order
5111241
@Iliyan-Y
User can see when the post is created
2f3b25b
@Iliyan-Y
Order test now check for date too
071e311
@Iliyan-Y
Test for user sign-up created
72df7e4
@Iliyan-Y
User fill form to create account
11516a9
@Iliyan-Y
Test for class user added
60dc59e
@Iliyan-Y
test for creating user passed
c4b8970
@Iliyan-Y
controller refactured to create new user
badcae4
@Iliyan-Y
User can log in feature test passed
d52376e
@Iliyan-Y
Log in test for class user added
b7913ca
@Iliyan-Y
Test for User class log in passed
58f2317
@Iliyan-Y
controller refactured to use model for log in
8782048
@Iliyan-Y
Test for error if user doesn't exist
92aa6da
@Iliyan-Y
Email verification added
bf0af4e
@Iliyan-Y
Test for password added
fb15ca6
@Iliyan-Y
Password test passed
d556ae5
@Iliyan-Y
Feature test for wrong email added
0a8e46f
@Iliyan-Y
Password and email verification added
0073c9c
@Iliyan-Y
Feature test added for user tring to register twice with the same email
15c8f01
@Iliyan-Y
Code refactured
0bd5a82
@Iliyan-Y
Code refactured
c704aea
@Iliyan-Y
Test for encrypting password added
023c7bc
@Iliyan-Y
Test passed for hashed password
71c34b0
@Iliyan-Y
Test for user buttons
a2341b8
@Iliyan-Y
Navigation user buttons added
9020f59
@michaelgroves90
Copy link

Nicely refactored code. Things have been kept DRY and skinny.

@michaelgroves90
Copy link

Instructions could be clearer and more in depth.

@michaelgroves90
Copy link

Some screenshots of pages to demonstrate use of CSS etc would be good to include

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Iliyan-Y @michaelgroves90