Skip to content
This repository has been archived by the owner on Nov 11, 2021. It is now read-only.

token existence validation #252

Merged
merged 1 commit into from
Nov 22, 2018
Merged

token existence validation #252

merged 1 commit into from
Nov 22, 2018

Conversation

szymonlesisz
Copy link
Contributor

Validation if selected token in send form is still assigned to seleted account.

This should fix below example:

  1. goto summary, add token
  2. goto send, create tx draft with this token
  3. go back to summary, remove this token
  4. go back to send (draft points to unsigned token - send form is broken)

@szymonlesisz szymonlesisz added bug Something isn't working request review Need someone to review a solution for an issue labels Nov 21, 2018
@szymonlesisz szymonlesisz changed the base branch from master to beta November 21, 2018 15:28
@szymonlesisz szymonlesisz changed the base branch from beta to master November 21, 2018 15:30
@szymonlesisz szymonlesisz self-assigned this Nov 21, 2018
@vladimirvolek vladimirvolek merged commit 35b97f9 into master Nov 22, 2018
@szymonlesisz szymonlesisz mentioned this pull request Nov 26, 2018
Merged
@szymonlesisz szymonlesisz deleted the fix/validate-token-in-send-draft branch December 4, 2018 18:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@vladimirvolek vladimirvolek Awaiting requested review from vladimirvolek

Assignees

@szymonlesisz szymonlesisz

Labels
bug Something isn't working request review Need someone to review a solution for an issue
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@szymonlesisz @vladimirvolek