-
Notifications
You must be signed in to change notification settings - Fork 14
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
Release version 4.20.0 #333
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add support for PDF reports
…-for-release-build Optimize GitHub workflow for release build
…mpatibility with dorny/test-reporter@v1 dorny/test-reporter#363
…-for-release-build Update actions to use Nodejs v20
…width-listeners-added Fix 11 auto-update-toast:width listeners added
…wner-for-auto-update-in-manual-run Add option to set repo owner for auto-update in manual run
Release version 4.20.0 to staging
ezewer
approved these changes
Mar 6, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm!
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
[4.20.0] - 2024-03-06
Added
2FA
. PR: bfx-report-ui#774Ledgers
andTax Reports
. PRs: bfx-report#347, bfx-reports-framework#352, bfx-report-electron#319, bfx-ext-pdf-js#4, bfx-report#349, bfx-reports-framework#354, bfx-report-ui#775BFX
auth error:ERR_AUTH_API: ERR_TOKEN_ALREADY_USED
. PR: bfx-report#348Changed
Movements
report according to the latest UX improvement proposals: remove tabs from wallets & movements, make movements a separate navigation item underMy History
. PR: bfx-report-ui#771My History
section according to the latest UX improvement proposals. PR: bfx-report-ui#778loading
andno data
states representation for reports with tables. PR: bfx-report-ui#77990sec
timeout for grc requests to have the same timeout as for api requests. PR: bfx-report#35190sec
timeout forhtml-pdf
lib. PR: bfx-reports-framework#355v20
. PR: bfx-report-electron#323Fixed
2FA
authorization flow according to: After the first push of the auth button, we should lock the button (till we get any response from this endpoint) to prevent sending several of the same requests. PR: bfx-report-ui#776OTP
: it should keep btn disabled until the successful auth will be completed. PR: bfx-report-ui#780auto-update-toast:width
listeners added. PR: bfx-report-electron#330