Skip to content

Alter the use of RemoteRepository #2759

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

Closed
3 tasks
agjohnson opened this issue Mar 28, 2017 · 2 comments
Closed
3 tasks

Alter the use of RemoteRepository #2759

agjohnson opened this issue Mar 28, 2017 · 2 comments
Labels
Accepted Accepted issue on our roadmap Improvement Minor improvement to code
Milestone

Comments

@agjohnson
Copy link
Contributor

Thinking about this more, RemoteRepository should not be a mapping of user access to remote repository objects, but instead should be just a list of accumulated remote repository objects. There should be a secondary table that provides a many to many mapping between remote users and remote repository objects. This lets us associate a project to a GitHub repo, for example, and multiple users would get access to the project on our side.

  • Add a model for remote user <-> remote repository
  • Add a migration that moves this data around
  • Make sync operations update both tables
@agjohnson agjohnson added Improvement Minor improvement to code Needed: design decision A core team decision is required labels Mar 28, 2017
@agjohnson agjohnson added this to the Admin UX milestone Sep 19, 2018
@agjohnson agjohnson added Accepted Accepted issue on our roadmap and removed Needed: design decision A core team decision is required labels Sep 19, 2018
@aswinshenoy
Copy link

I wish to work this issue @agjohnson, is this still open?
Regards

@humitos
Copy link
Member

humitos commented Feb 23, 2021

This was done in originally #7536, and the work continued in the branch https://github.com/readthedocs/readthedocs.org/tree/remote-repository-normalization

I think we can close this issue. I'm trying to target the start of data sync for next deploy.

@humitos humitos closed this as completed Feb 23, 2021
astrojuanlu pushed a commit to astrojuanlu/rtd-dev-docs that referenced this issue Oct 27, 2021
Initial document to discuss the path forward to implement
readthedocs/readthedocs.org#2759
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Accepted Accepted issue on our roadmap Improvement Minor improvement to code
Projects
None yet
Development

No branches or pull requests

3 participants