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

need confirmation number to give to frontend #106

Closed
lihanli opened this issue Sep 15, 2016 · 3 comments · Fixed by #161
Closed

need confirmation number to give to frontend #106

lihanli opened this issue Sep 15, 2016 · 3 comments · Fixed by #161
Assignees
@lihanli
Labels
ed benefits

Comments

@lihanli
Copy link
Contributor

lihanli commented Sep 15, 2016

i'm thinking about adding a uuid column to education_benefits_claims and using that for the confirmation number, thoughts @markolson?
@lihanli lihanli self-assigned this Sep 15, 2016
@markolson
Copy link
Contributor

Let's just use the EducationBenefitsClaim id.

@markolson
Copy link
Contributor

I'm going to assign this to myself as I'm working in that area right now as well.

@markolson markolson self-assigned this Sep 15, 2016
@lihanli lihanli removed their assignment Sep 15, 2016
@akainic
Copy link
Contributor

akainic commented Sep 26, 2016 •
edited

We're going to use the id from our database table. We need to add to serialized output.

@lihanli lihanli assigned lihanli and unassigned markolson Sep 26, 2016
@lihanli lihanli mentioned this issue Sep 26, 2016
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lihanli lihanli

Labels
ed benefits
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add confirmation number department-of-veterans-affairs/vets-api
3 participants
@markolson @lihanli @akainic