Skip to content

User can log out#9

Open
Alex-Bellamy wants to merge 19 commits intoCraftAcademy:masterfrom
Alex-Bellamy:user_can_log_out
Open

User can log out#9
Alex-Bellamy wants to merge 19 commits intoCraftAcademy:masterfrom
Alex-Bellamy:user_can_log_out

Conversation

@Alex-Bellamy
Copy link
Copy Markdown
Member

@Alex-Bellamy Alex-Bellamy commented Jun 27, 2020

https://www.pivotaltracker.com/story/show/173478558

User Story

As a user

In order to exit the site

I want be able to log out

Changes proposed in this pull request:

  • Specs confirming a user can log out

What I have learned working on this feature:

  • Warden function

Odeane and others added 19 commits June 24, 2020 15:27
@Odeane
fixing gem bug, creating rspec folder and creating db migration to co…
6913aac 
…rrect format
@Alex-Bellamy
installing Rspec, creating a database, updating migration version, ap…
c1720bd 
…p now loads on server
created a specfolder and updated the db migration version to 4.2 and …
76f7174 
…created a db migartion for add_missing_indices.mailboxer_engine
@Alex-Bellamy
login functionality - adding test to ensure Login button is present a…
2583dc9 
…nd the button functions which pass
wrote teo new tests for log in feature, first i changed root_path to …
2368b8d 
…user_session_path then wrote test to locate Email to make sure i was in the right path second test was for filling in email, password and then clickon log in button
writing test for redirecting efter log in
62fbc43
updated README
d3d1558
@Odeane
testing setup
83b73b6
@Odeane
setting up factor bot, adding shoulder matcher
d909ed5
@Alex-Bellamy
Merge branch 'log_in_feature' of https://github.com/Odeane/legacy_cod…
02562d0 
…e_june_2020
@viktorsommar
Adding coverage files to git ignore
dfc8403
@Alex-Bellamy
adding branch sending_message_feature
9fb149f
@viktorsommar
Pulling changes from alex branch.
e493a4e
@Cma-B
start working on sending message
4e24b0c
@Cma-B
edits gemfile
434329e
@viktorsommar
Adding tests for happy path message sending. Switching programer in t…
55c2f82 
…eam.
@Cma-B
removes merge confilict
3648e75
@Cma-B
writes test for sending message, add factory bot for recipient, makes…
c8126fd 
… model and controller for recipient
@Alex-Bellamy
adding user can log out spec, tests successfully passes
93d66f3
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.

4 participants

@Alex-Bellamy @Odeane @viktorsommar @Cma-B