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

Transaction Page: automated tests #3518

Merged
merged 3 commits into from
Jul 27, 2021
Merged

Transaction Page: automated tests #3518

merged 3 commits into from
Jul 27, 2021

Conversation

bgptr
Copy link
Collaborator

@bgptr bgptr commented Jun 25, 2021

This is the continuation of #3493. I've covered the Transaction Page view:

  • app/components/views/TransactionPage/*
  • app/components/views/TransactionPage/TransactionContent/*
  • app/components/views/TransactionPage/TransactionHeader/*

I've fixed also:

  • the header title and icon of missed, live, unmined, and immature ticket pages. They were titled simply by Ticket with default ticket icons.
  • the layout of the pending transaction view on the smaller screens. The buttons were overlapped:
    DeepinScreenshot_select-area_20210624144920

@alexlyp
Copy link
Member

alexlyp commented Jul 13, 2021

Needs a quick rebase

@bgptr
Copy link
Collaborator Author

bgptr commented Jul 13, 2021

@alexlyp rebased

@alexlyp alexlyp merged commit 785c40b into decred:master Jul 27, 2021
@bgptr bgptr deleted the test-transactionPage branch August 27, 2021 16:23
@bgptr bgptr mentioned this pull request Oct 25, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@matheusd matheusd matheusd approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@bgptr @alexlyp @matheusd