Skip to content
This repository has been archived by the owner on Jan 7, 2024. It is now read-only.

Updates tests and data file for first and last name #92

Merged
merged 1 commit into from
Jun 18, 2019
Merged

Conversation

kushaldas
Copy link
Contributor

Our API already provides first and last name for the current
user, now we have tests for the same.

How to test?

make test should do.

Our API already provides first and last name for the current
user, now we have tests for the same.
@kushaldas kushaldas requested a review from sssoleileraaa June 13, 2019 16:31
Copy link
Contributor

@sssoleileraaa sssoleileraaa left a comment

Choose a reason for hiding this comment

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

this lgtm

As follow-up, we will need to use get_current_user in the client after a successful authenticate to get the first and last name and update the user table in the local db.

@sssoleileraaa
Copy link
Contributor

Oh wait, nevermind the follow-up would be adding first and last name to the existing users table.

@sssoleileraaa
Copy link
Contributor

Closes #87

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants