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.

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

sabau/1681-Modal proposal feedback #1692

Merged
merged 2 commits into from
Dec 4, 2018

Conversation

sabau
Copy link
Contributor

@sabau sabau commented Dec 4, 2018

PageProposal:

  • Fix votingStartedAgo attribute from voting_start_block to voting_start_time

ModalPropose:

  • error messages for title and description maxLength
  • error class enabled in form group if the data is not validated
  • short form methods for validators

Closes #1681
Contribute to #1690

❤️ Thank you!


  • Added entries in CHANGELOG.md with issue # and GitHub username
  • Reviewed Files changed in the github PR explorer

- Fix votingStartedAgo attribute from voting_start_block to voting_start_time

ModalPropose:
- error messages for title and description maxLength
- error class enabled in form group if the data is not validated
- short form methods for validators

Signed-off-by: Karoly Albert Szabo <[email protected]>
@sabau sabau requested review from faboweb and nylira as code owners December 4, 2018 14:10
@codecov
Copy link

codecov bot commented Dec 4, 2018

Codecov Report

Merging #1692 into develop will decrease coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           develop   #1692      +/-   ##
==========================================
- Coverage    97.71%   97.7%   -0.01%     
==========================================
  Files          101     101              
  Lines         2056    2052       -4     
  Branches        92      92              
==========================================
- Hits          2009    2005       -4     
  Misses          36      36              
  Partials        11      11
Impacted Files Coverage Δ
...rc/renderer/components/governance/ModalPropose.vue 95.23% <100%> (-0.77%) ⬇️
...rc/renderer/components/governance/PageProposal.vue 100% <100%> (ø) ⬆️

1 similar comment
@codecov
Copy link

codecov bot commented Dec 4, 2018

Codecov Report

Merging #1692 into develop will decrease coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           develop   #1692      +/-   ##
==========================================
- Coverage    97.71%   97.7%   -0.01%     
==========================================
  Files          101     101              
  Lines         2056    2052       -4     
  Branches        92      92              
==========================================
- Hits          2009    2005       -4     
  Misses          36      36              
  Partials        11      11
Impacted Files Coverage Δ
...rc/renderer/components/governance/ModalPropose.vue 95.23% <100%> (-0.77%) ⬇️
...rc/renderer/components/governance/PageProposal.vue 100% <100%> (ø) ⬆️

@sabau sabau requested a review from fedekunze December 4, 2018 14:11
@fedekunze fedekunze changed the title Modal proposal feedback sabau/1681-Modal proposal feedback Dec 4, 2018
@jbibla jbibla merged commit d9250a7 into develop Dec 4, 2018
@jbibla jbibla deleted the sabau/1681-governance-error-feedbacks branch December 4, 2018 15:23
@faboweb
Copy link
Collaborator

faboweb commented Dec 4, 2018

amazing, already the 2nd PR!

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.

Proposal Creation feedbacks
3 participants