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

Feat: #751 show all workspaces to admin #820

Draft
wants to merge 24 commits into
base: master
Choose a base branch
from

Conversation

marikaris
Copy link
Collaborator

@marikaris marikaris commented Dec 10, 2024

New PR because old one got tangled up with the docs PR for some reason.

how to test:

  • explain here what to do to test this (or point to unit tests)

todo:

  • Endpoint for displaying workspaces of all users
  • Save new workspaces in directory with user email, so we know which workspaces belong to what user
  • Upon saving workspace, check if workspaces available saved by uuid, if so copy them to folder with email address
  • Delete endpoint for workspaces from admin perspective
  • Load workspaces from new bucket directory
  • UI for displaying and deleting workspaces
  • Tests
  • See if it's possible to add file to new bucket if contents have been migrated

@marikaris
Copy link
Collaborator Author

Ran into this bug:
#821

The delete workspace endpoint has the same problem. I think this bug needs to be fixed in this PR as well.

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

Successfully merging this pull request may close these issues.

2 participants