Feat: Added support for multiple project owners #4597

aryan-bhokare · 2024-04-24T06:22:38Z

Proposed changes

Added new Route '/projects'.
Added ProjectDashboard.
Modified API's

Types of changes

What types of changes does your code introduce to Litmus? Put an x in the boxes that apply

New feature (non-breaking change which adds functionality)
Bugfix (non-breaking change which fixes an issue)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
Documentation Update (if none of the other choices applies)

Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

I have read the CONTRIBUTING doc
I have signed the commit for DCO to be passed.
Lint and unit tests pass locally with my changes
I have added tests that prove my fix is effective or that my feature works (if appropriate)
I have added necessary documentation (if appropriate)

Dependency

Pr #4536

Special notes for your reviewer:

Test Scenarios

Project can have multiple project owners so checking RBACs by switching users(owners).
Can invite with the owner role.
User cannot change his own role to avoid last owner changing his role.
User cannot remove invitation of themselves to avoid the last owner to leave.
Last project owner leaves.
Owner can remove other owners.

Signed-off-by: aryan <aryan1bhokare@gmail.com>

…itmuschaos#4527) Bumps [golang.org/x/crypto](https://github.com/golang/crypto) from 0.18.0 to 0.21.0. - [Commits](golang/crypto@v0.18.0...v0.21.0) --- updated-dependencies: - dependency-name: golang.org/x/crypto dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

) Bumps [follow-redirects](https://github.com/follow-redirects/follow-redirects) from 1.15.5 to 1.15.6. - [Release notes](https://github.com/follow-redirects/follow-redirects/releases) - [Commits](follow-redirects/follow-redirects@v1.15.5...v1.15.6) --- updated-dependencies: - dependency-name: follow-redirects dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bumps [github.com/golang/protobuf](https://github.com/golang/protobuf) from 1.5.3 to 1.5.4. - [Release notes](https://github.com/golang/protobuf/releases) - [Commits](golang/protobuf@v1.5.3...v1.5.4) --- updated-dependencies: - dependency-name: github.com/golang/protobuf dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Raj Das <mail.rajdas@gmail.com>

This modification unables to send invite with the role as owner. Signed-off-by: aryan <aryan1bhokare@gmail.com>

This modification checks if the User is the last owner of the project and if not User can leave the project. Signed-off-by: aryan <aryan1bhokare@gmail.com>

Allows Owner to be able to leave the project. Signed-off-by: aryan <aryan1bhokare@gmail.com>

This API is used for updating role of the member in the project. Signed-off-by: aryan <aryan1bhokare@gmail.com>

Signed-off-by: aryan <aryan1bhokare@gmail.com>

Owner can delete project with help of this API. Signed-off-by: aryan <aryan1bhokare@gmail.com>

Signed-off-by: aryan <aryan1bhokare@gmail.com>

…kend Signed-off-by: Aryan Bhokare <92683836+aryan-bhokare@users.noreply.github.com>

Signed-off-by: aryan <aryan1bhokare@gmail.com>

User cannot change role of their own, so that it will avoid edge cases like 1. User is the last owner of the project. 2. User accidentally losing owner access to the projects. Signed-off-by: aryan <aryan1bhokare@gmail.com>

Signed-off-by: aryan <aryan1bhokare@gmail.com>

* Modified db schema of Owner. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added new API GetProjectOwners. Signed-off-by: aryan <aryan1bhokare@gmail.com> * fix: return type error. Signed-off-by: aryan <aryan1bhokare@gmail.com> * chore(deps): Bump golang.org/x/crypto in /chaoscenter/authentication (litmuschaos#4527) Bumps [golang.org/x/crypto](https://github.com/golang/crypto) from 0.18.0 to 0.21.0. - [Commits](golang/crypto@v0.18.0...v0.21.0) --- updated-dependencies: - dependency-name: golang.org/x/crypto dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * chore(deps): Bump follow-redirects in /chaoscenter/web (litmuschaos#4529) Bumps [follow-redirects](https://github.com/follow-redirects/follow-redirects) from 1.15.5 to 1.15.6. - [Release notes](https://github.com/follow-redirects/follow-redirects/releases) - [Commits](follow-redirects/follow-redirects@v1.15.5...v1.15.6) --- updated-dependencies: - dependency-name: follow-redirects dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * chore(deps): Bump github.com/golang/protobuf (litmuschaos#4493) Bumps [github.com/golang/protobuf](https://github.com/golang/protobuf) from 1.5.3 to 1.5.4. - [Release notes](https://github.com/golang/protobuf/releases) - [Commits](golang/protobuf@v1.5.3...v1.5.4) --- updated-dependencies: - dependency-name: github.com/golang/protobuf dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Raj Das <mail.rajdas@gmail.com> * Modified SendInvitation API. This modification unables to send invite with the role as owner. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Modified LeaveProject API. This modification checks if the User is the last owner of the project and if not User can leave the project. Signed-off-by: aryan <aryan1bhokare@gmail.com> * RBAC modification `LeaveProject`. Allows Owner to be able to leave the project. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added `UpdateMemberRole` API. This API is used for updating role of the member in the project. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Fixed some syntax errors. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Updated roles for owner. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added new API `DeleteProject`. Owner can delete project with help of this API. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added mocks. Signed-off-by: aryan <aryan1bhokare@gmail.com> * modified go.sum Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added condition `UpdateMemberRole`. User cannot change role of their own, so that it will avoid edge cases like 1. User is the last owner of the project. 2. User accidentally losing owner access to the projects. Signed-off-by: aryan <aryan1bhokare@gmail.com> * made suggested changes. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Changed DeleteProject endpoint to have url parameter. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Minor fixes. Signed-off-by: aryan <aryan1bhokare@gmail.com> --------- Signed-off-by: aryan <aryan1bhokare@gmail.com> Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: Aryan Bhokare <92683836+aryan-bhokare@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Raj Das <mail.rajdas@gmail.com>

Signed-off-by: aryan <aryan1bhokare@gmail.com>

Added a modal CreateProject with it's controller and views. Signed-off-by: aryan <aryan1bhokare@gmail.com>

Signed-off-by: aryan <aryan1bhokare@gmail.com>

aryan-bhokare · 2024-04-24T06:24:01Z

@hrishavjha @SahilKr24 @Saranya-jena PTAL :)

chaoscenter/web/src/components/ProjectDashboardCardContainer/ProjectDashboardCardContainer.tsx

chaoscenter/web/src/controllers/ProjectDashboardCardMenu/ProjectDashboardCardMenu.tsx

chaoscenter/web/src/views/CreateProject/CreateProject.tsx

chaoscenter/web/src/controllers/ProjectDashboard/ProjectFilters.tsx

SahilKr24

Nothing that seems worthy of a change request. General feedback and some optimisations.

Code is clean, precise and less than 128 lines per file. Looks good to me. Great job.

chaoscenter/web/src/components/ProjectDashboardCardContainer/ProjectDashboardCardContainer.tsx

chaoscenter/web/src/components/SideNav/SideNav.tsx

chaoscenter/web/src/controllers/ProjectDashboard/ProjectDashboard.tsx

chaoscenter/web/src/views/ProjectDashboard/ProjectDashboard.tsx

chaoscenter/web/src/views/ProjectDashboardCardMenu/ProjectDashboardCardMenu.tsx

hrishavjha

A Few minor improvements can be made as mentioned in the comments, rest of the changes LGTM 🚀

…s.tsx Co-authored-by: Hrishav <hrishav.kumar@harness.io> Signed-off-by: Aryan Bhokare <92683836+aryan-bhokare@users.noreply.github.com>

Signed-off-by: aryan <aryan1bhokare@gmail.com>

chaoscenter/authentication/pkg/project/repository.go

chaoscenter/authentication/pkg/utils/project_utils.go

chaoscenter/authentication/pkg/project/repository.go

SarthakJain26 · 2024-06-04T08:41:50Z

chaoscenter/authentication/pkg/entities/project.go

+	Projects              []*Project `json:"projects"`
+	TotalNumberOfProjects *int64     `json:"totalNumberOfProjects"`


Both the fields can be made mandatory, since these fields will always be returned in the response. Empty array can be returned in projects in case no projects are found

chaoscenter/authentication/pkg/project/repository.go

chaoscenter/authentication/pkg/utils/project_utils.go

Signed-off-by: aryan <aryan1bhokare@gmail.com>

Signed-off-by: Aryan Bhokare <92683836+aryan-bhokare@users.noreply.github.com>

Signed-off-by: aryan <aryan1bhokare@gmail.com>

* Modified db schema of Owner. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added new API GetProjectOwners. Signed-off-by: aryan <aryan1bhokare@gmail.com> * fix: return type error. Signed-off-by: aryan <aryan1bhokare@gmail.com> * chore(deps): Bump golang.org/x/crypto in /chaoscenter/authentication (litmuschaos#4527) Bumps [golang.org/x/crypto](https://github.com/golang/crypto) from 0.18.0 to 0.21.0. - [Commits](golang/crypto@v0.18.0...v0.21.0) --- updated-dependencies: - dependency-name: golang.org/x/crypto dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * chore(deps): Bump follow-redirects in /chaoscenter/web (litmuschaos#4529) Bumps [follow-redirects](https://github.com/follow-redirects/follow-redirects) from 1.15.5 to 1.15.6. - [Release notes](https://github.com/follow-redirects/follow-redirects/releases) - [Commits](follow-redirects/follow-redirects@v1.15.5...v1.15.6) --- updated-dependencies: - dependency-name: follow-redirects dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * chore(deps): Bump github.com/golang/protobuf (litmuschaos#4493) Bumps [github.com/golang/protobuf](https://github.com/golang/protobuf) from 1.5.3 to 1.5.4. - [Release notes](https://github.com/golang/protobuf/releases) - [Commits](golang/protobuf@v1.5.3...v1.5.4) --- updated-dependencies: - dependency-name: github.com/golang/protobuf dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Raj Das <mail.rajdas@gmail.com> * Modified SendInvitation API. This modification unables to send invite with the role as owner. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Modified LeaveProject API. This modification checks if the User is the last owner of the project and if not User can leave the project. Signed-off-by: aryan <aryan1bhokare@gmail.com> * RBAC modification `LeaveProject`. Allows Owner to be able to leave the project. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added `UpdateMemberRole` API. This API is used for updating role of the member in the project. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Fixed some syntax errors. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Updated roles for owner. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added new API `DeleteProject`. Owner can delete project with help of this API. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added mocks. Signed-off-by: aryan <aryan1bhokare@gmail.com> * modified go.sum Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added condition `UpdateMemberRole`. User cannot change role of their own, so that it will avoid edge cases like 1. User is the last owner of the project. 2. User accidentally losing owner access to the projects. Signed-off-by: aryan <aryan1bhokare@gmail.com> * made suggested changes. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Changed DeleteProject endpoint to have url parameter. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Minor fixes. Signed-off-by: aryan <aryan1bhokare@gmail.com> * [WIP] : Multiple project owner backend. (litmuschaos#4536) * Modified db schema of Owner. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added new API GetProjectOwners. Signed-off-by: aryan <aryan1bhokare@gmail.com> * fix: return type error. Signed-off-by: aryan <aryan1bhokare@gmail.com> * chore(deps): Bump golang.org/x/crypto in /chaoscenter/authentication (litmuschaos#4527) Bumps [golang.org/x/crypto](https://github.com/golang/crypto) from 0.18.0 to 0.21.0. - [Commits](golang/crypto@v0.18.0...v0.21.0) --- updated-dependencies: - dependency-name: golang.org/x/crypto dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * chore(deps): Bump follow-redirects in /chaoscenter/web (litmuschaos#4529) Bumps [follow-redirects](https://github.com/follow-redirects/follow-redirects) from 1.15.5 to 1.15.6. - [Release notes](https://github.com/follow-redirects/follow-redirects/releases) - [Commits](follow-redirects/follow-redirects@v1.15.5...v1.15.6) --- updated-dependencies: - dependency-name: follow-redirects dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * chore(deps): Bump github.com/golang/protobuf (litmuschaos#4493) Bumps [github.com/golang/protobuf](https://github.com/golang/protobuf) from 1.5.3 to 1.5.4. - [Release notes](https://github.com/golang/protobuf/releases) - [Commits](golang/protobuf@v1.5.3...v1.5.4) --- updated-dependencies: - dependency-name: github.com/golang/protobuf dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Raj Das <mail.rajdas@gmail.com> * Modified SendInvitation API. This modification unables to send invite with the role as owner. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Modified LeaveProject API. This modification checks if the User is the last owner of the project and if not User can leave the project. Signed-off-by: aryan <aryan1bhokare@gmail.com> * RBAC modification `LeaveProject`. Allows Owner to be able to leave the project. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added `UpdateMemberRole` API. This API is used for updating role of the member in the project. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Fixed some syntax errors. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Updated roles for owner. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added new API `DeleteProject`. Owner can delete project with help of this API. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added mocks. Signed-off-by: aryan <aryan1bhokare@gmail.com> * modified go.sum Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added condition `UpdateMemberRole`. User cannot change role of their own, so that it will avoid edge cases like 1. User is the last owner of the project. 2. User accidentally losing owner access to the projects. Signed-off-by: aryan <aryan1bhokare@gmail.com> * made suggested changes. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Changed DeleteProject endpoint to have url parameter. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Minor fixes. Signed-off-by: aryan <aryan1bhokare@gmail.com> --------- Signed-off-by: aryan <aryan1bhokare@gmail.com> Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: Aryan Bhokare <92683836+aryan-bhokare@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Raj Das <mail.rajdas@gmail.com> * Added new route . Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added `CreateProject` modal. Added a modal CreateProject with it's controller and views. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Some changes in `CreateProjectView`. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added `ProjectDashboardCardMenu`. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added `ProjectDashboardCard`. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added `DeleteProject` API mutations. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added `ProjectDashboard`. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added image and strings. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Modified `project entities`. Added new fields in `Project` struct. Added fields for filters, pagination, and some constants. Modified `CreateProjectInput`. Signed-off-by: aryan <aryan1bhokare@gmail.com> * [Backend] Modification in Backend for the UI. Added Filters and pagination in Backend. Modified API's and added a pipeline for the aggregation of results. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added `project_util` for validation of input request. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Indent Fixes Signed-off-by: aryan <aryan1bhokare@gmail.com> * Modification for Frontend Hook of `CreateProject` API. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Modified `ListProject` Query frontend hook. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Removed string constants and some minor changes. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added Project Filters. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added pagination and filter subheader in Dashboard. Signed-off-by: aryan <aryan1bhokare@gmail.com> * modified auth-api swagger file. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added tags section in create-project modal. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Changes due to modification of API and addition of new strings Signed-off-by: aryan <aryan1bhokare@gmail.com> * minor changes and resolved some errors. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added routing when clicked on the card. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Modifications in backend tests as per API updates. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Fix: NoProjects Element and NoFilteredProject Results element. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added scroll for the project list. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Some changes in UI w.r.t Multiple Project Owner Feature. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Made search text type insensitive. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Update chaoscenter/web/src/controllers/ProjectDashboard/ProjectFilters.tsx Co-authored-by: Hrishav <hrishav.kumar@harness.io> Signed-off-by: Aryan Bhokare <92683836+aryan-bhokare@users.noreply.github.com> * requested changes. Signed-off-by: aryan <aryan1bhokare@gmail.com> * removed unnecessary handle function Signed-off-by: aryan <aryan1bhokare@gmail.com> * requested backend changes and small fixes Signed-off-by: aryan <aryan1bhokare@gmail.com> * Changed folder structure. Signed-off-by: aryan <aryan1bhokare@gmail.com> * requested changes. Signed-off-by: aryan <aryan1bhokare@gmail.com> * fixed import orders Signed-off-by: aryan <aryan1bhokare@gmail.com> * fixing RoleEditor to RoleExecuter Signed-off-by: aryan <aryan1bhokare@gmail.com> * removed redundant deleteprojectinput Signed-off-by: aryan <aryan1bhokare@gmail.com> * fixed bug caused in merging Signed-off-by: aryan <aryan1bhokare@gmail.com> * fixed bug caused in merging Signed-off-by: aryan <aryan1bhokare@gmail.com> * removed duplicate struct Signed-off-by: aryan <aryan1bhokare@gmail.com> * Fix: frontend chaoshub test Signed-off-by: aryan <aryan1bhokare@gmail.com> * fix: tag rendering issue in project dashboard Signed-off-by: aryan <aryan1bhokare@gmail.com> * fix: Less user details in createProject Signed-off-by: aryan <aryan1bhokare@gmail.com> * fix: import orders Signed-off-by: aryan <aryan1bhokare@gmail.com> --------- Signed-off-by: aryan <aryan1bhokare@gmail.com> Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: Aryan Bhokare <92683836+aryan-bhokare@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Raj Das <mail.rajdas@gmail.com> Co-authored-by: Hrishav <hrishav.kumar@harness.io> Co-authored-by: Saranya Jena <saranya.jena@harness.io> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

* Modified db schema of Owner. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added new API GetProjectOwners. Signed-off-by: aryan <aryan1bhokare@gmail.com> * fix: return type error. Signed-off-by: aryan <aryan1bhokare@gmail.com> * chore(deps): Bump golang.org/x/crypto in /chaoscenter/authentication (litmuschaos#4527) Bumps [golang.org/x/crypto](https://github.com/golang/crypto) from 0.18.0 to 0.21.0. - [Commits](golang/crypto@v0.18.0...v0.21.0) --- updated-dependencies: - dependency-name: golang.org/x/crypto dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * chore(deps): Bump follow-redirects in /chaoscenter/web (litmuschaos#4529) Bumps [follow-redirects](https://github.com/follow-redirects/follow-redirects) from 1.15.5 to 1.15.6. - [Release notes](https://github.com/follow-redirects/follow-redirects/releases) - [Commits](follow-redirects/follow-redirects@v1.15.5...v1.15.6) --- updated-dependencies: - dependency-name: follow-redirects dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * chore(deps): Bump github.com/golang/protobuf (litmuschaos#4493) Bumps [github.com/golang/protobuf](https://github.com/golang/protobuf) from 1.5.3 to 1.5.4. - [Release notes](https://github.com/golang/protobuf/releases) - [Commits](golang/protobuf@v1.5.3...v1.5.4) --- updated-dependencies: - dependency-name: github.com/golang/protobuf dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Raj Das <mail.rajdas@gmail.com> * Modified SendInvitation API. This modification unables to send invite with the role as owner. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Modified LeaveProject API. This modification checks if the User is the last owner of the project and if not User can leave the project. Signed-off-by: aryan <aryan1bhokare@gmail.com> * RBAC modification `LeaveProject`. Allows Owner to be able to leave the project. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added `UpdateMemberRole` API. This API is used for updating role of the member in the project. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Fixed some syntax errors. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Updated roles for owner. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added new API `DeleteProject`. Owner can delete project with help of this API. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added mocks. Signed-off-by: aryan <aryan1bhokare@gmail.com> * modified go.sum Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added condition `UpdateMemberRole`. User cannot change role of their own, so that it will avoid edge cases like 1. User is the last owner of the project. 2. User accidentally losing owner access to the projects. Signed-off-by: aryan <aryan1bhokare@gmail.com> * made suggested changes. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Changed DeleteProject endpoint to have url parameter. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Minor fixes. Signed-off-by: aryan <aryan1bhokare@gmail.com> * [WIP] : Multiple project owner backend. (litmuschaos#4536) * Modified db schema of Owner. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added new API GetProjectOwners. Signed-off-by: aryan <aryan1bhokare@gmail.com> * fix: return type error. Signed-off-by: aryan <aryan1bhokare@gmail.com> * chore(deps): Bump golang.org/x/crypto in /chaoscenter/authentication (litmuschaos#4527) Bumps [golang.org/x/crypto](https://github.com/golang/crypto) from 0.18.0 to 0.21.0. - [Commits](golang/crypto@v0.18.0...v0.21.0) --- updated-dependencies: - dependency-name: golang.org/x/crypto dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * chore(deps): Bump follow-redirects in /chaoscenter/web (litmuschaos#4529) Bumps [follow-redirects](https://github.com/follow-redirects/follow-redirects) from 1.15.5 to 1.15.6. - [Release notes](https://github.com/follow-redirects/follow-redirects/releases) - [Commits](follow-redirects/follow-redirects@v1.15.5...v1.15.6) --- updated-dependencies: - dependency-name: follow-redirects dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * chore(deps): Bump github.com/golang/protobuf (litmuschaos#4493) Bumps [github.com/golang/protobuf](https://github.com/golang/protobuf) from 1.5.3 to 1.5.4. - [Release notes](https://github.com/golang/protobuf/releases) - [Commits](golang/protobuf@v1.5.3...v1.5.4) --- updated-dependencies: - dependency-name: github.com/golang/protobuf dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Raj Das <mail.rajdas@gmail.com> * Modified SendInvitation API. This modification unables to send invite with the role as owner. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Modified LeaveProject API. This modification checks if the User is the last owner of the project and if not User can leave the project. Signed-off-by: aryan <aryan1bhokare@gmail.com> * RBAC modification `LeaveProject`. Allows Owner to be able to leave the project. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added `UpdateMemberRole` API. This API is used for updating role of the member in the project. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Fixed some syntax errors. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Updated roles for owner. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added new API `DeleteProject`. Owner can delete project with help of this API. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added mocks. Signed-off-by: aryan <aryan1bhokare@gmail.com> * modified go.sum Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added condition `UpdateMemberRole`. User cannot change role of their own, so that it will avoid edge cases like 1. User is the last owner of the project. 2. User accidentally losing owner access to the projects. Signed-off-by: aryan <aryan1bhokare@gmail.com> * made suggested changes. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Changed DeleteProject endpoint to have url parameter. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Minor fixes. Signed-off-by: aryan <aryan1bhokare@gmail.com> --------- Signed-off-by: aryan <aryan1bhokare@gmail.com> Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: Aryan Bhokare <92683836+aryan-bhokare@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Raj Das <mail.rajdas@gmail.com> * Added new route . Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added `CreateProject` modal. Added a modal CreateProject with it's controller and views. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Some changes in `CreateProjectView`. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added `ProjectDashboardCardMenu`. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added `ProjectDashboardCard`. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added `DeleteProject` API mutations. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added `ProjectDashboard`. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added image and strings. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Modified `project entities`. Added new fields in `Project` struct. Added fields for filters, pagination, and some constants. Modified `CreateProjectInput`. Signed-off-by: aryan <aryan1bhokare@gmail.com> * [Backend] Modification in Backend for the UI. Added Filters and pagination in Backend. Modified API's and added a pipeline for the aggregation of results. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added `project_util` for validation of input request. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Indent Fixes Signed-off-by: aryan <aryan1bhokare@gmail.com> * Modification for Frontend Hook of `CreateProject` API. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Modified `ListProject` Query frontend hook. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Removed string constants and some minor changes. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added Project Filters. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added pagination and filter subheader in Dashboard. Signed-off-by: aryan <aryan1bhokare@gmail.com> * modified auth-api swagger file. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added tags section in create-project modal. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Changes due to modification of API and addition of new strings Signed-off-by: aryan <aryan1bhokare@gmail.com> * minor changes and resolved some errors. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added routing when clicked on the card. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Modifications in backend tests as per API updates. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Fix: NoProjects Element and NoFilteredProject Results element. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Added scroll for the project list. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Some changes in UI w.r.t Multiple Project Owner Feature. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Made search text type insensitive. Signed-off-by: aryan <aryan1bhokare@gmail.com> * Update chaoscenter/web/src/controllers/ProjectDashboard/ProjectFilters.tsx Co-authored-by: Hrishav <hrishav.kumar@harness.io> Signed-off-by: Aryan Bhokare <92683836+aryan-bhokare@users.noreply.github.com> * requested changes. Signed-off-by: aryan <aryan1bhokare@gmail.com> * removed unnecessary handle function Signed-off-by: aryan <aryan1bhokare@gmail.com> * requested backend changes and small fixes Signed-off-by: aryan <aryan1bhokare@gmail.com> * Changed folder structure. Signed-off-by: aryan <aryan1bhokare@gmail.com> * requested changes. Signed-off-by: aryan <aryan1bhokare@gmail.com> * fixed import orders Signed-off-by: aryan <aryan1bhokare@gmail.com> * fixing RoleEditor to RoleExecuter Signed-off-by: aryan <aryan1bhokare@gmail.com> * removed redundant deleteprojectinput Signed-off-by: aryan <aryan1bhokare@gmail.com> * fixed bug caused in merging Signed-off-by: aryan <aryan1bhokare@gmail.com> * fixed bug caused in merging Signed-off-by: aryan <aryan1bhokare@gmail.com> * removed duplicate struct Signed-off-by: aryan <aryan1bhokare@gmail.com> * Fix: frontend chaoshub test Signed-off-by: aryan <aryan1bhokare@gmail.com> * fix: tag rendering issue in project dashboard Signed-off-by: aryan <aryan1bhokare@gmail.com> * fix: Less user details in createProject Signed-off-by: aryan <aryan1bhokare@gmail.com> * fix: import orders Signed-off-by: aryan <aryan1bhokare@gmail.com> --------- Signed-off-by: aryan <aryan1bhokare@gmail.com> Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: Aryan Bhokare <92683836+aryan-bhokare@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Raj Das <mail.rajdas@gmail.com> Co-authored-by: Hrishav <hrishav.kumar@harness.io> Co-authored-by: Saranya Jena <saranya.jena@harness.io> Signed-off-by: sagnik3788 <116512372+sagnik3788@users.noreply.github.com>

aryan-bhokare and others added 29 commits March 17, 2024 12:19

Modified db schema of Owner.

78b82ab

Signed-off-by: aryan <aryan1bhokare@gmail.com>

Added new API GetProjectOwners.

962175b

Signed-off-by: aryan <aryan1bhokare@gmail.com>

fix: return type error.

0c8528e

Signed-off-by: aryan <aryan1bhokare@gmail.com>

Modified SendInvitation API.

1de73da

This modification unables to send invite with the role as owner. Signed-off-by: aryan <aryan1bhokare@gmail.com>

Modified LeaveProject API.

7c79813

This modification checks if the User is the last owner of the project and if not User can leave the project. Signed-off-by: aryan <aryan1bhokare@gmail.com>

RBAC modification LeaveProject.

44ae3b9

Allows Owner to be able to leave the project. Signed-off-by: aryan <aryan1bhokare@gmail.com>

Added UpdateMemberRole API.

77bcd91

This API is used for updating role of the member in the project. Signed-off-by: aryan <aryan1bhokare@gmail.com>

Fixed some syntax errors.

43bd3b2

Signed-off-by: aryan <aryan1bhokare@gmail.com>

Updated roles for owner.

2ac9f7d

Signed-off-by: aryan <aryan1bhokare@gmail.com>

Added new API DeleteProject.

1860721

Owner can delete project with help of this API. Signed-off-by: aryan <aryan1bhokare@gmail.com>

Added mocks.

a2a27d4

Signed-off-by: aryan <aryan1bhokare@gmail.com>

Merge branch 'multiple-owner-support' into multiple-project-owner-bac…

4e8279a

…kend Signed-off-by: Aryan Bhokare <92683836+aryan-bhokare@users.noreply.github.com>

modified go.sum

737c03f

Signed-off-by: aryan <aryan1bhokare@gmail.com>

Added condition UpdateMemberRole.

c1b2ed3

User cannot change role of their own, so that it will avoid edge cases like 1. User is the last owner of the project. 2. User accidentally losing owner access to the projects. Signed-off-by: aryan <aryan1bhokare@gmail.com>

made suggested changes.

197aabd

Signed-off-by: aryan <aryan1bhokare@gmail.com>

Changed DeleteProject endpoint to have url parameter.

4a10fe3

Signed-off-by: aryan <aryan1bhokare@gmail.com>

Minor fixes.

33dec51

Signed-off-by: aryan <aryan1bhokare@gmail.com>

Added new route .

3a4ee69

Signed-off-by: aryan <aryan1bhokare@gmail.com>

Added CreateProject modal.

d2d8f17

Added a modal CreateProject with it's controller and views. Signed-off-by: aryan <aryan1bhokare@gmail.com>

Some changes in CreateProjectView.

e9d81ad

Signed-off-by: aryan <aryan1bhokare@gmail.com>

Added ProjectDashboardCardMenu.

e0b4518

Signed-off-by: aryan <aryan1bhokare@gmail.com>

Added ProjectDashboardCard.

43bdca2

Signed-off-by: aryan <aryan1bhokare@gmail.com>

Added DeleteProject API mutations.

d368644

Signed-off-by: aryan <aryan1bhokare@gmail.com>

Added ProjectDashboard.

409beb9

Signed-off-by: aryan <aryan1bhokare@gmail.com>

Added image and strings.

28b890c

Signed-off-by: aryan <aryan1bhokare@gmail.com>

hrishavjha requested changes May 22, 2024

View reviewed changes

hrishavjha reviewed May 22, 2024

View reviewed changes

chaoscenter/web/src/controllers/ProjectDashboard/ProjectFilters.tsx Outdated Show resolved Hide resolved

SahilKr24 approved these changes May 22, 2024

View reviewed changes

hrishavjha approved these changes May 25, 2024

View reviewed changes

aryan-bhokare and others added 4 commits May 25, 2024 15:19

Update chaoscenter/web/src/controllers/ProjectDashboard/ProjectFilter…

554e008

…s.tsx Co-authored-by: Hrishav <hrishav.kumar@harness.io> Signed-off-by: Aryan Bhokare <92683836+aryan-bhokare@users.noreply.github.com>

requested changes.

42da32e

Signed-off-by: aryan <aryan1bhokare@gmail.com>

removed unnecessary handle function

4dbd103

Signed-off-by: aryan <aryan1bhokare@gmail.com>

requested backend changes and small fixes

aa157f0

Signed-off-by: aryan <aryan1bhokare@gmail.com>

SarthakJain26 requested changes Jun 4, 2024

View reviewed changes

aryan-bhokare and others added 8 commits June 13, 2024 17:50

Changed folder structure.

92a2273

Signed-off-by: aryan <aryan1bhokare@gmail.com>

requested changes.

6daa086

Signed-off-by: aryan <aryan1bhokare@gmail.com>

Merge branch 'master' into multiple-project-owner-backend

0fe519a

Signed-off-by: Aryan Bhokare <92683836+aryan-bhokare@users.noreply.github.com>

fixed import orders

f7062d5

Signed-off-by: aryan <aryan1bhokare@gmail.com>

fixing RoleEditor to RoleExecuter

a381cb4

Signed-off-by: aryan <aryan1bhokare@gmail.com>

merge backend and frontend branch

b034c7c

Signed-off-by: aryan <aryan1bhokare@gmail.com>

removed redundant deleteprojectinput

11837fd

Signed-off-by: aryan <aryan1bhokare@gmail.com>

fixed bug caused in merging

d0d4e1e

Signed-off-by: aryan <aryan1bhokare@gmail.com>

aryan-bhokare changed the base branch from multiple-owner-support to master July 18, 2024 11:20

Saranya-jena and others added 8 commits July 22, 2024 11:48

Merge branch 'master' into multiple-project-owner-frontend

e52d7ba

Merge branch 'master' into multiple-project-owner-frontend

3a6f762

fixed bug caused in merging

f7a4612

Signed-off-by: aryan <aryan1bhokare@gmail.com>

removed duplicate struct

2b0c246

Signed-off-by: aryan <aryan1bhokare@gmail.com>

Fix: frontend chaoshub test

615f705

Signed-off-by: aryan <aryan1bhokare@gmail.com>

fix: tag rendering issue in project dashboard

006af80

Signed-off-by: aryan <aryan1bhokare@gmail.com>

fix: Less user details in createProject

ecbeb76

Signed-off-by: aryan <aryan1bhokare@gmail.com>

fix: import orders

fe85c89

Signed-off-by: aryan <aryan1bhokare@gmail.com>

Saranya-jena merged commit 9d09211 into litmuschaos:master Jul 22, 2024

		Projects []*Project `json:"projects"`
		TotalNumberOfProjects *int64 `json:"totalNumberOfProjects"`

Feat: Added support for multiple project owners #4597

Feat: Added support for multiple project owners #4597

Uh oh!

Conversation

aryan-bhokare commented Apr 24, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Proposed changes

Types of changes

Checklist

Dependency

Special notes for your reviewer:

Test Scenarios

Uh oh!

aryan-bhokare commented Apr 24, 2024

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

SahilKr24 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

hrishavjha left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

SarthakJain26 Jun 4, 2024

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

aryan-bhokare commented Apr 24, 2024 •

edited

Loading