这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@ZIMkaRU
Copy link
Member

@ZIMkaRU ZIMkaRU commented Mar 29, 2023

[4.4.0] - 2023-03-29

Added

  • Implemented username/password auth support with Two-Factor Authentication for simple users in a framework mode and corresponding logic to handle various Reports authorization flow-related specifics. Improved Reports login modal styling according to the actual theme. PR: bfx-report-ui#622
  • Added ability to auto-start sync after the auto-update of the electron app. The idea is to have a configurable option (by default turned on) to force sync after the auto-update or DB migration due to changes in the DB schema, the aim is to bring data consistency after significant updates. PRs: bfx-report-electron#190, bfx-reports-framework#261
  • Added isAuthTokenGenerationError: true flag into the Unauthorized 401 response in cases when token is expired for better UX of 2FA. PRs: bfx-report#285, bfx-reports-framework#262
  • Added login/verify proxy endpoints to resolve the CORS issue for the BFX username/password auth for /v2/login and /v2/login/verify links. PRs: bfx-reports-framework#263, bfx-report#286

Changed

  • Changed the Sign Up section title and button to Add Account for better clearance to the users of how auth flow works in the Reports app. Hid the Remove Account button in the Preferences menu during syncing to avoid causing related errors. Prevented the Remember Me from being prefilled by default. Updated passwords titles. PR: bfx-report-ui#623
  • Fixed the getUsers endpoint response, isRestrictedToBeAddedToSubAccount flag doesn't show the correct state, it should be true in a case when the user signed in with the BFX auth token (using BFX username/password). PR: bfx-reports-framework#260

Copy link
Contributor

@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 fc70c16 into master Mar 29, 2023
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.

4 participants