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

Release version 4.28.0 #409

Merged
merged 4 commits into from
Sep 24, 2024
Merged

Release version 4.28.0 #409

merged 4 commits into from
Sep 24, 2024

Conversation

ZIMkaRU
Copy link
Member

@ZIMkaRU ZIMkaRU commented Sep 24, 2024

[4.28.0] - 2024-09-25

Added

  • Implemented dynamic height calculation for the Concentration Risk pie chart to prevent overflow issues possibility spotted in some cases. PR: bfx-report-ui#859
  • Implemented Last Sync time handling and representation (approximately in hours) for the Reports. PR: bfx-report-ui#863
  • Implemented Profits section (port of the Win/Loss chart with several predefined parameters) on the app Summary page. Removed charts smoothness for better precision. PR: bfx-report-ui#864
  • Added logic to have separated translations by language in JSON files using i18next lib for easier translation maintenance. PRs: bfx-report#402, bfx-reports-framework#417

Changed

  • Improved user notification and auth flow behavior for the cases when the user tries to re-add an existing account via email/password. PR: bfx-report-ui#860

Fixed

  • Improved Docker/Terraform deployment, fixed html-pdf module usage under Docker container with using docker container based on the Debian image to make html-pdf module workable, fixed deprecation warnings. PR: bfx-reports-framework#415
  • Fixed an infrequent case for process.send() when the app is on its way to being closed and the child process channel is closed but the worker still sends a message to the main one. PR: bfx-reports-framework#416

Security

  • Bumped path-to-regexp from 1.8.0 to 1.9.0, express from 4.19.2 to 4.21.0. PR: bfx-report-ui#858

Copy link
Collaborator

@ezewer ezewer left a comment

Choose a reason for hiding this comment

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

lgtm!

@ezewer ezewer merged commit 4884884 into master Sep 24, 2024
2 checks passed
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.

2 participants