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.

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

Connect removing a site from a project to backend #433

Closed
Tracked by #30
ltseng opened this issue Sep 18, 2023 · 5 comments · Fixed by #552
Closed
Tracked by #30

Connect removing a site from a project to backend #433

ltseng opened this issue Sep 18, 2023 · 5 comments · Fixed by #552

Comments

@ltseng
Copy link
Contributor

ltseng commented Sep 18, 2023

Should probably have a warning dialog

This specifically refers to the "remove from project" option when selecting the three dots/More icon on a site card within the project's Sites tab.

@ltseng ltseng added the landpks label Sep 18, 2023
@ltseng ltseng added this to the LandPKS 2023-09 milestone Sep 18, 2023
@ltseng
Copy link
Contributor Author

ltseng commented Sep 18, 2023

@CourtneyLee333 do you have a warning message in mind?

@CourtneyLee333
Copy link

@ltseng I haven't written one. Do we need one? Will the site retain its membership when it is removed from a project? What are the consequences for different team roles of removing the site from the project?

@ltseng
Copy link
Contributor Author

ltseng commented Sep 18, 2023

When a project manager removes a site from a project, but doesn't delete it, they basically transfer ownership of the site to themselves. As a result, all other project members will lose access to that site.

@CourtneyLee333
Copy link

Is it an official decision that sites will not have team members, just the single site owner? If so, the dialog can be:

Remove site from project?
This action will remove other team members from the site, and make the site accessible only to you. Any previously collected site data will be retained.

@ltseng
Copy link
Contributor Author

ltseng commented Sep 18, 2023

It's not an official one yet, more one that we're hoping we can sidestep by implementing project management very well, and the pure fact of the matter being that we simply won't have time to get to it. I think this works for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants