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

Fix broken export to CSV #3009

Merged
merged 1 commit into from
Feb 14, 2020
Merged

Conversation

vbaranov
Copy link
Member

@vbaranov vbaranov commented Feb 14, 2020

Motivation

Export to CSV button returns redirect to not found page. It was broken in #2834

Changelog

add checksummed hash in address_id param for transactions_csv method

Checklist for your Pull Request (PR)

@vbaranov vbaranov self-assigned this Feb 14, 2020
@vbaranov vbaranov force-pushed the vb-fix-redirect-to-checksummed-address branch from 7f354d5 to 39e1817 Compare February 14, 2020 15:52
@vbaranov vbaranov force-pushed the vb-fix-redirect-to-checksummed-address branch from 39e1817 to fdf8b6a Compare February 14, 2020 15:57
@vbaranov vbaranov added the ready for review This PR is ready for reviews. label Feb 14, 2020
@coveralls
Copy link

coveralls commented Feb 14, 2020

Pull Request Test Coverage Report for Build 749cf47a-4cad-4d04-bbb7-db41ed4e101b

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 75.402%

Totals Coverage Status
Change from base Build 1d67edc3-b799-4fc7-a46f-526569bc43d0: 0.0%
Covered Lines: 5438
Relevant Lines: 7212

💛 - Coveralls

@vbaranov vbaranov merged commit f704982 into master Feb 14, 2020
@vbaranov vbaranov deleted the vb-fix-redirect-to-checksummed-address branch February 14, 2020 16:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready for review This PR is ready for reviews.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants