Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Make the Application Security Team on the report dynamic #52

Closed
alejandrosaenz117 opened this issue May 19, 2020 · 0 comments
Closed

Make the Application Security Team on the report dynamic #52

alejandrosaenz117 opened this issue May 19, 2020 · 0 comments
Assignees
@alejandrosaenz117
Labels
enhancement New feature or request
Milestone
Q2 Goals

Comments

@alejandrosaenz117
Copy link
Collaborator

The Application Security Team in the generated report is hardcoded. Now that user information has been implemented, it's time to make the team dynamic. The assessment form will have a new pick-list where the user can choose who the testers will be. These testers will then be listed on the generated report.
@alejandrosaenz117 alejandrosaenz117 added the enhancement New feature or request label May 19, 2020
@alejandrosaenz117 alejandrosaenz117 added this to the Q2 Goals milestone May 19, 2020
@alejandrosaenz117 alejandrosaenz117 self-assigned this May 19, 2020
alejandrosaenz117 added a commit that referenced this issue May 21, 2020
@alejandrosaenz117
feat(assessment form and report): dynamic tester association to asssment
3bbfc9c 
Added a ManyToMany relation between the User and Assessment models.  Added a muliple select option
to the assessment.component.  Added APIs to get users.  Patched the create and update assessment
APIs.  Updated the report to retrieve testers.

BREAKING CHANGE: ManyToMany relationship has been created between the User and Assessment models.
API's have been updated for this change.  New API's created to retrieve users.

feat #52
alejandrosaenz117 added a commit that referenced this issue May 22, 2020
@alejandrosaenz117
refactor(assessment controller): updated response message. Removed usrId
c2d2555 
Updated response messages to say request and not UserRequest.  Removed userId from select statement
for report.

re #52
@alejandrosaenz117 alejandrosaenz117 mentioned this issue May 22, 2020
Merged
18 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@alejandrosaenz117 alejandrosaenz117

Labels
enhancement New feature or request
Projects
None yet
Milestone
Q2 Goals
Development

No branches or pull requests
1 participant
@alejandrosaenz117