Skip to content

[WIP] payments/funds-recipient/personal-id-number component - #895

Closed
pixelhandler wants to merge 8 commits into
code-corps:developfrom
pixelhandler:886_payments/funds-recipient/personal-id-number_component
Closed

[WIP] payments/funds-recipient/personal-id-number component#895
pixelhandler wants to merge 8 commits into
code-corps:developfrom
pixelhandler:886_payments/funds-recipient/personal-id-number_component

Conversation

@pixelhandler

@pixelhandler pixelhandler commented Dec 18, 2016

Copy link
Copy Markdown
Contributor

What's in this PR?

  • WIP for new payments/funds-recipient/personal-id-number component

References

Progress on: #886

@pixelhandler

Copy link
Copy Markdown
Contributor Author

This is only work in progress, will squash later when closer to complete

@begedin begedin assigned begedin and unassigned begedin Dec 19, 2016
@pixelhandler

pixelhandler commented Dec 19, 2016

Copy link
Copy Markdown
Contributor Author

There is more work still to be done, I think the logic isn't right for showing the verifying message, prolly needs to be there with a disabled submit button. I'm TDD'ing so that will be the next step.

I pushed up a copy of my fork/branch upstream as well: https://github.com/code-corps/code-corps-ember/tree/886_payments/funds-recipient/personal-id-number_component

@joshsmith joshsmith added this to the Launch Donations milestone Dec 20, 2016
@begedin

begedin commented Dec 22, 2016

Copy link
Copy Markdown
Contributor

@pixelhandler This is the last bit of the puzzle, really, feature-wise, so I'll take over and wrap up today. Good work so far.

@begedin begedin self-assigned this Dec 22, 2016
@pixelhandler

Copy link
Copy Markdown
Contributor Author

@begedin ok thanks!

@joshsmith

Copy link
Copy Markdown
Contributor

Think this was superseded by #906.

@joshsmith joshsmith closed this Dec 30, 2016
@joshsmith joshsmith removed this from the Launch Donations milestone Dec 30, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants