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

fix(ui): center failed payment error modal dialog #3188

Merged
merged 1 commit into from
Nov 15, 2019
Merged

fix(ui): center failed payment error modal dialog #3188

merged 1 commit into from
Nov 15, 2019

Conversation

bolatovumar
Copy link
Contributor

Motivation and Context:

Fix #3183

How Has This Been Tested?

Manually

Screenshots (if appropriate):

Screen Shot 2019-11-14 at 7 31 24 PM

Checklist:

  • My code follows the code style of this project.
  • I have reviewed and updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes where needed.
  • All new and existing tests passed.
  • My commits have been squashed into a concise set of changes.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 22.783% when pulling b08353c on bolatovumar:fix-3183 into d800a01 on LN-Zap:master.

mrfelton
mrfelton approved these changes Nov 15, 2019
View reviewed changes
Copy link
Member

@mrfelton mrfelton left a comment

Choose a reason for hiding this comment

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

Tested ACK b08353c

@mrfelton mrfelton merged commit b089312 into LN-Zap:master Nov 15, 2019
@mrfelton mrfelton added the type: bug 🐛 Something isn't working label Nov 15, 2019
@mrfelton mrfelton added this to the v0.6.0-beta milestone Nov 15, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@mrfelton mrfelton mrfelton approved these changes

Assignees

@bolatovumar bolatovumar

Labels
type: bug 🐛 Something isn't working
Projects
None yet
Milestone
v0.6.0-beta
Development

Successfully merging this pull request may close these issues.

Transaction error modal dialog positioning of off-center
3 participants
@bolatovumar @coveralls @mrfelton