topcoder-platform / micro-frontends-taas-admin-app

taas-admin-app for managing taas administrative activities
MIT License
0 stars 5 forks source link

Simple CRUD interface for Role Management #111

Closed wdprice closed 3 years ago

wdprice commented 3 years ago

Build a simple CRUD tool inside the TaaS Admin tab of platform.topcoder.com which allows admins to edit the taas-role data.

Must allow creation of a new role, and deletion of an existing role.

Following fields are must haves:

nikolay83 commented 3 years ago

@wdprice

Must allow creation of a new role, and deletion of an existing role

I assume editing a role is also required given that each role will have 10+ attributes.

wdprice commented 3 years ago

Yes that is correct

On Fri, Jul 23, 2021 at 4:40 PM Nikolay Iakovlev @.***> wrote:

@wdprice https://github.com/wdprice

Must allow creation of a new role, and deletion of an existing role

I assume editing a role is also required given that each role will have 10+ attributes.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/topcoder-platform/taas-app/issues/403#issuecomment-885902530, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABN4IYP7QCSNVPKLFUSWVQ3TZHHT5ANCNFSM5A2ZNCJA .

wdprice commented 3 years ago

@urwithat @nikolay83 - is this one under way?

nikolay83 commented 3 years ago

@wdprice yes, I expect it today.

sdgun commented 3 years ago

Verified in Dev. Reported the below issues.

https://github.com/topcoder-platform/taas-app/issues/426 https://github.com/topcoder-platform/taas-app/issues/425 https://github.com/topcoder-platform/taas-app/issues/423 https://github.com/topcoder-platform/taas-app/issues/422 https://github.com/topcoder-platform/taas-app/issues/421

wdprice commented 3 years ago

Couple of questions on this:

How does the multiple rate card features work? Rate 1, Rate 2, etc?

Can we update the number fields to accept typed values instead of forcing the user to use the incremental +/- buttons?

What about making those strings so we can specify? FOr example, if we want "TIme to Candidates" to say 1 week, would be good to be able to put the word "week" in there so the user understands the frame of reference.

cagdas001 commented 3 years ago

@wdprice

How does the multiple rate card features work? Rate 1, Rate 2, etc?

The app. can work with multiple rates (i.e. backend accepts rates as array). So each rate card represents one individual rate

Can we update the number fields to accept typed values instead of forcing the user to use the incremental +/- buttons?

Tracked here: topcoder-platform/micro-frontends-taas-admin-app#107

What about making those strings so we can specify? FOr example, if we want "TIme to Candidates" to say 1 week, would be good to be able to put the word "week" in there so the user understands the frame of reference.

To be clear, you want to see units (week, hours etc.) as well alongside with the fields? Or you want user to be able to provide units? (like entering "3 weeks" or "2 hours" in fields)

wdprice commented 3 years ago

Providing it in fields works too.

On Wed, Aug 4, 2021 at 3:20 PM Cagdas U. @.***> wrote:

@wdprice https://github.com/wdprice

How does the multiple rate card features work? Rate 1, Rate 2, etc?

The app. can work with multiple rates (i.e. backend accepts rates as array). So each rate card represents one individual rate

Can we update the number fields to accept typed values instead of forcing the user to use the incremental +/- buttons?

Tracked here: topcoder-platform/micro-frontends-taas-admin-app#107 https://github.com/topcoder-platform/taas-app/issues/425

What about making those strings so we can specify? FOr example, if we want "TIme to Candidates" to say 1 week, would be good to be able to put the word "week" in there so the user understands the frame of reference.

To be clear, you want to see units (week, hours etc.) as well alongside with the fields? Or you want user to be able to provide units? (like entering "3 weeks" or "2 hours" in fields)

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/topcoder-platform/taas-app/issues/403#issuecomment-892910675, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABN4IYMCMPWDHXHUCH2ALU3T3GHHTANCNFSM5A2ZNCJA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email .

sdgun commented 3 years ago

Verified in Dev, all issues and requested changes have been addressed. As of now one related issue is reported https://github.com/topcoder-platform/taas-app/issues/439

lakshmiathreya commented 3 years ago
Screenshot 2021-08-16 at 1 42 20 PM Screenshot 2021-08-16 at 1 51 01 PM Screenshot 2021-08-16 at 1 57 42 PM
lakshmiathreya commented 3 years ago

-- Create and Edit of Role is verified -- Default Rate is set to 10 -- Global Rates- removal is being allowed Pls clarify @nikolay83 @sdgun

cagdas001 commented 3 years ago

Default rate should be set to 1000. Make sure the latest taas-admin-app is deployed to production

sdgun commented 3 years ago

Verified in production.

Screenshot 2021-08-16 at 3 32 46 PM
lakshmiathreya commented 3 years ago

-- Delete and Sync verified in Prod.