Nikodem Graczewski

OLMIS-6020: Added FunctionDecorator class
OLMIS-6020: Added FunctionDecorator class
OLMIS-6020: Extended the OpenlmisResource with the ability to pass custom parameters when calling create method.

OLMIS-6020: Removed ddescribe

OLMIS-6020: Added FunctionDecorator class

OLMIS-6020: Added Notification Settings page to the UserProfile with static list of digest configurations

    • -0
    • +51
    /src/openlmis-user/digest-configuration-data-builder.spec.js
    • -0
    • +43
    /src/openlmis-user/digest-configuration-resource.js
    • -0
    • +39
    /src/openlmis-user/digest-configuration-resource.spec.js
    • -0
    • +72
    /src/openlmis-user/notification-tag.filter.js
    • -0
    • +46
    /src/openlmis-user/notification-tag.filter.spec.js
    • -0
    • +62
    /src/openlmis-user/user-profile-notification-settings.controller.js
    • -0
    • +45
    /src/openlmis-user/user-profile-notification-settings.controller.spec.js
    • -0
    • +18
    /src/openlmis-user/user-profile-notification-settings.html
    • -0
    • +14
    /src/openlmis-user/user-profile.routes.js
OLMIS-6020: Updated URL pointing to user profile page

OLMIS-6020: Added message for User Profile string

OLMIS-6071: Fixed a bug with navigationStateService evaluating for parent states with child states...
OLMIS-6071: Fixed a bug with navigationStateService evaluating for parent states with child states...
OLMIS-6071: Fixed a bug with navigationStateService evaluating for parent states with child states doing asynchronous canAccess calls

OLMIS-6020: Reworked user profile into multiple tabs
OLMIS-6020: Reworked user profile into multiple tabs
Fair enough.

Fair enough.

Could we modify this part that it returns the map so we don't have to check for the flag in multiple other places?

Could we modify this part that it returns the map so we don't have to check for the flag in multiple other places?

OLMIS-6020: Fixed invalid file name

    • -224
    • +0
    /src/openlmis-user/user-profile-basic-information.controller.spec..js
    • -0
    • +224
    /src/openlmis-user/user-profile-basic-information.controller.spec.js
OLMIS-6020: Updated link to default user profile page

OLMIS-6020: Reworked user profile into multiple tabs

    • -1
    • +1
    /src/admin-user-roles/user-role-tabs.html
    • -0
    • +178
    /src/openlmis-user/user-profile-basic-information.controller.js
    • -0
    • +224
    /src/openlmis-user/user-profile-basic-information.controller.spec..js
    • -0
    • +51
    /src/openlmis-user/user-profile-basic-information.html
    • -0
    • +77
    /src/openlmis-user/user-profile-role-assignments.controller.js
    • -0
    • +51
    /src/openlmis-user/user-profile-role-assignments.controller.spec.js
    • -0
    • +3
    /src/openlmis-user/user-profile-role-assignments.html
    • -53
    • +9
    /src/openlmis-user/user-profile.html
    • -46
    • +61
    /src/openlmis-user/user-profile.routes.js
OLMIS-6073: Fixed a bug with current user cache not being properly cleared

Feel free to close the review.

Feel free to close the review.

OLMIS-6071: Refactored requisition-approval.routes.spec.js
OLMIS-6071: Refactored requisition-approval.routes.spec.js
OLMIS-6071: Made Approve Requisitions screen accessible to partner users
OLMIS-6071: Made Approve Requisitions screen accessible to partner users
OLMIS-6071: Made Approve Requisitions screen accessible to partner users

I don't think we can do that as we still have one unresolved comment and the changes requested are not yet implemented.

I don't think we can do that as we still have one unresolved comment and the changes requested are not yet implemented.

Could we change to something about reducing the number of requests?

Could we change to something about reducing the number of requests?

OLMIS-6071: Refactored requisition-approval.routes.spec.js