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

Fix repo permissions and only sync on demand #485

Open
1 of 3 tasks
6543 opened this issue Oct 27, 2021 · 2 comments
Open
1 of 3 tasks

Fix repo permissions and only sync on demand #485

6543 opened this issue Oct 27, 2021 · 2 comments
Labels
enhancement improve existing features
Milestone

Comments

@6543
Copy link
Member

6543 commented Oct 27, 2021

Instead of pull all permission data of all repos for a user periodicaly, only check and update if needed.

(User request info for a repo ... and permission check is out of sync)

ttl of permission data should be settable too

TODOs:

  • revert Add flag for not fetching permissions (FlatPermissions) (#491) #625
  • clarify permission model (currently 3 props pull, push, admin) and public repos currently get pull as fallback
    instead we should make clear a user has explicit pull request (member of the project) or is just able to see data but is not a member (this would fix some cases where users see repos not owned in their UI)
  • ...
@6543 6543 added the enhancement improve existing features label Oct 27, 2021
@6543 6543 added this to the 0.15.0 milestone Oct 30, 2021
@6543 6543 modified the milestones: 0.15.0, 0.16.0 Dec 19, 2021
@6543
Copy link
Member Author

6543 commented Dec 19, 2021

will frontport #491 ... since milestone changed :/

@anbraten anbraten changed the title Sync repo permissions on demand Fix repo permissions and only sync on demand Mar 1, 2022
@6543
Copy link
Member Author

6543 commented Sep 1, 2022

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement improve existing features
Projects
None yet
Development

No branches or pull requests

2 participants