Skip to content
This repository was archived by the owner on Mar 13, 2025. It is now read-only.

Ownership transfer #43

Closed
cwdcwd opened this issue Aug 10, 2018 · 3 comments
Closed

Ownership transfer #43

cwdcwd opened this issue Aug 10, 2018 · 3 comments
Labels
enhancement New feature or request P3 Critical but non-blocking

Comments

@cwdcwd
Copy link
Contributor

cwdcwd commented Aug 10, 2018

It would be convenient for an Admin to set up a project and then transfer project ownership to another user, such as a copilot.

UI

In the UI, for an admin, we will show a new option on the Edit screen for a project. This button will be Transfer ownership and will be in the header of the screen.

When Transfer ownership is clicked, we will show a popup with this information:

  • Title: Transfer ownership
  • A text field labeled Topcoder handle
  • OK button
  • Cancel button

When OK is clicked, we will validate that the Topcoder handle field is filled.

Functionality

When the OK button is clicked, we will:

  1. Check the user mappings to ensure we have a mapping for the new user
  2. Update the owner of the repo with the new topcoder handle.

If we don't have a mapping for the new user, we need to ensure that an error message is shown to the user saying User <TC Handle> doesn't currently have Topcoder-X access. Please have them sign in and set up their Gitlab and Github accounts with Topcoder-X before transferring ownership.

@cwdcwd cwdcwd added the enhancement New feature or request label Aug 10, 2018
@cwdcwd
Copy link
Contributor Author

cwdcwd commented Aug 13, 2018

@jmgasper any thoughts on this one?

@cwdcwd cwdcwd added the P3 Critical but non-blocking label Aug 13, 2018
@jmgasper
Copy link
Collaborator

I think that would work, but the new user will have to have access to the project in Gitlab / Github or things might break. I'm not sure how to do that test yet.

@cwdcwd
Copy link
Contributor Author

cwdcwd commented Aug 16, 2018

well to be robust we'd check our db of user mappings and block if the user wasn't in there I suppose

jmgasper added a commit that referenced this issue Sep 21, 2018
#44
#61 (Major requirement)
#42 (Major requirement)
#43
#77 (Major requirement)
#66
jmgasper added a commit that referenced this issue Sep 25, 2018
#44
#61 (Major requirement)
#42 (Major requirement)
#43
#77 (Major requirement)
#66
jmgasper added a commit that referenced this issue Oct 9, 2019
#44
#61 (Major requirement)
#42 (Major requirement)
#43
#77 (Major requirement)
#66
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request P3 Critical but non-blocking
Projects
None yet
Development

No branches or pull requests

2 participants