Skip to content

Fixes method for retrieving user by email#810

Merged
amoose merged 1 commit intomasterfrom
amos-fix-user_flow_specs
Dec 5, 2016
Merged

Fixes method for retrieving user by email#810
amoose merged 1 commit intomasterfrom
amos-fix-user_flow_specs

Conversation

@amoose
Copy link
Contributor

@amoose amoose commented Dec 5, 2016

Why

User flow specs error when looking up user by email

How

Updates method to retrieve email since the encryption updates

Copy link
Contributor

@pkarman pkarman left a comment

Choose a reason for hiding this comment

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

lgtm


it 'prompts the user to log in or sign up' do
screenshot_and_save_page
screenshot_and_open_image
Copy link
Contributor

Choose a reason for hiding this comment

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

Did you mean to leave this here or was it just for debugging?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm working with Adam to include screenshots in the output; I left this one here while we work on that output.

@amoose amoose merged commit 38b2057 into master Dec 5, 2016
@amoose amoose deleted the amos-fix-user_flow_specs branch December 5, 2016 21:32
amoose added a commit that referenced this pull request Feb 24, 2017
amoose added a commit that referenced this pull request Feb 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants