Skip to content
This repository was archived by the owner on Mar 13, 2025. It is now read-only.

fix(roles): roles page bugs #85

Merged
merged 3 commits into from
Aug 4, 2021
Merged

Conversation

cagdas001
Copy link
Contributor

  • (b11eb0d) fix(roles): reset modalError when re-opened
    • The modalError of the previous operation was still being displayed
      even after closing & opening the modal again.

    • This fix resets the modalError each time it re-opened.

      Addresses topcoder-platform/taas-app#423

  • (6c79c7a) fix(roles): typo in the role creation notification
    • Fix a typo in the "Successfully craeted the role." message
      of role creation notification.

    • Replace craeted -> created

      Addresses topcoder-platform/taas-app#422

  • (870b4d5) fix(roles): NaN rate values in the RoleForm
    • Fix a bug with the IntegerField controls of the RoleForm,
      that was occurring when user trying to increase/decrease
      a number field which initially has null.

    • This was causing NaN values in the field.

    • This fix starts IntegerField controls with 0,
      in case the user is trying to increase/decrease a null field.

      Addresses topcoder-platform/taas-app#421

* The `modalError` of the previous operation was still being displayed
  even after closing & opening the modal again.
* This fix resets the `modalError` each time it re-opened.

Addresses https://github.com/topcoder-platform/taas-app/issues/423
* Fix a typo in the "Successfully craeted the role." message
  of role creation notification.
  * Replace `craeted` -> `created`

Addresses https://github.com/topcoder-platform/taas-app/issues/422
* Fix a bug with the `IntegerField` controls of the `RoleForm`,
  that was occurring when user trying to increase/decrease
  a number field which initially has `null`.
* This was causing `NaN` values in the field.
* This fix starts `IntegerField` controls with `0`,
  in case the user is trying to increase/decrease a `null` field.

Addresses https://github.com/topcoder-platform/taas-app/issues/421
@urwithat urwithat merged commit d99814d into topcoder-archive:dev Aug 4, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants