CSUMB-SP17-CST499 / Group-Assignment-Tool

0 stars 3 forks source link

Add action button to Employee, Add authentication to the action button, Fixed Role add, Connected Role delete -Stephan #131

Closed thedirtyham closed 7 years ago

thedirtyham commented 7 years ago

Created the action button and linked it to adding new roles to an employee, created authentication to use the action button. You now must be logged in to use the action button. Fixed the endpoint in roles/put that was causing an error for adding roles. Connected the Delete role function to the correct button and current API endpoint call.

TEST PLAN Make sure you can see the action button on the employee's page by logging in. Make sure you ca add a new role to an existing employee. Add a new role Attach new group to role Delete new role

thedirtyham commented 7 years ago

On the roles page when you click on the action button and then on the delete button the action button disappears. No other section appears. Low key confused what you are saying.

eliasargandara commented 7 years ago

Sorry, I must have been tired. I meant the Employee page. Here's a picture of what I was talking about. There's a selection menu for both actions at the same time. It should only be possible to have one open at a time. employees_page_screenshot

thedirtyham commented 7 years ago

the button that goes away is in the roles page, my button does not go away when you click the remove roles. I merged sals code and pushed the combine changes.