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

Backend/309/ Initial User Roles and Permissions #319

Closed
wants to merge 95 commits into from
Closed

Backend/309/ Initial User Roles and Permissions #319

wants to merge 95 commits into from

Conversation

RafterGit
Copy link

@RafterGit RafterGit commented Oct 9, 2020 •
edited by carltonsmith
Loading

This pull request changes...

[AC] This pull request is ready to merge when...

  • Tests have been updated (and all tests are passing)
  • Snyk.io has no high/critcial CVEs
  • Linting Tests (ESLint for UI and Flake8 for Python)
  • Unit Tests ( including a code coverage test report)
  • This code has been reviewed by someone other than the original author
  • The change has been documented
  • Associated OpenAPI documentation has been updated

Notes:

@RafterGit RafterGit added the WIP label Oct 9, 2020
carltonsmith
carltonsmith approved these changes Oct 20, 2020
View reviewed changes
Copy link

@carltonsmith carltonsmith left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me

@RafterGit RafterGit linked an issue Oct 21, 2020 that may be closed by this pull request
As an OFA Admin, I need a list of MVP roles in the database to assign to new users #309
Closed
4 tasks
@RafterGit RafterGit removed the WIP label Oct 22, 2020
@RafterGit RafterGit marked this pull request as ready for review October 22, 2020 14:46
carltonsmith and others added 18 commits December 30, 2020 12:28
@carltonsmith
Merge branch 'raft-tdp-main' into backend/309/dynamic-roles-permissions
d18ca97
@carltonsmith
Merge branch 'raft-tdp-main' into backend/309/dynamic-roles-permissions
2ca0d81
@carltonsmith
customize admin user form
6d00410
@carltonsmith
cleanup 309
5b0ddbd
@lfrohlich
Merge pull request #113 from raft-tech/snyk-and-security-updates
113e82b 
Snyk and security updates - With @alexsoble 's approval, I am merging this PR
@lfrohlich
Merge pull request #112 from raft-tech/documentation/282/session-mana…
7a72660 
…gement

Documentation/282/session management looks good
@carltonsmith
update docs
bc5c05f
@carltonsmith
remove todo
f3fb7bb
@carltonsmith
remove dup
acac761
@riatzukiza
Merge branch 'raft-tdp-main' into backend/309/dynamic-roles-permissions
cf3cccc
@carltonsmith
update docs
3909e9c
@carltonsmith
remove unneeded doc
bf7870c
@carltonsmith
add comment note
33f6277
@carltonsmith
typos
5b218cb
@carltonsmith
breakout user api tests
6f71f78
@carltonsmith
update permissions
1c5b4f8
@carltonsmith
linting
1e03954
@carltonsmith
Merge branch 'main' into backend/309/dynamic-roles-permissions
6b69d10
@carltonsmith carltonsmith added QASP Review and removed raft review This issue is ready for raft review labels Jan 5, 2021
@carltonsmith
Copy link

Closing as branch was updated with other branches

@andrew-jameson andrew-jameson deleted the backend/309/dynamic-roles-permissions branch July 27, 2021 13:36
riatzukiza pushed a commit that referenced this pull request Jul 22, 2022
@ADPennington
Merge pull request #319 from raft-tech/raft-tdp-main
68fe7b9 
Multi-Issue PR
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@carltonsmith carltonsmith carltonsmith approved these changes

@riatzukiza riatzukiza Awaiting requested review from riatzukiza

@alexsoble alexsoble Awaiting requested review from alexsoble

Assignees
No one assigned
Labels
QASP Review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

As an OFA Admin, I need a list of MVP roles in the database to assign to new users
6 participants
@RafterGit @alexsoble @carltonsmith @riatzukiza @hilvitzs @lfrohlich