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.

Sign up for GitHub

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

Jump to bottom

[Multi-Tenant] Allow viewing deleted tenants #3256

Closed
2 tasks done
Tracked by #2893
mkurapov opened this issue Jan 30, 2025 · 2 comments · Fixed by #3299
Closed
2 tasks done
Tracked by #2893

[Multi-Tenant] Allow viewing deleted tenants #3256

mkurapov opened this issue Jan 30, 2025 · 2 comments · Fixed by #3299
Assignees
@koekiebox
Labels
type: enhancement New feature or request

Comments

@mkurapov
Copy link
Contributor

mkurapov commented Jan 30, 2025 •
edited
Loading

Context

Currently, if a tenant is deleted, it is not possible to fetch it using the tenant query.
However, this doesn't play nicely with the Admin UI: you are unable to navigate to the tenant details screen from the tenant list page.

If you are the operator tenant, we should allow fetching deleted tenants, and being able to view details.

Todos

  • Allow the tenant Admin API resolver to return deleted tenants if an operator is making the request.
  • Allow clicking into tenant details in the Admin UI for deleted tenants. The fields should not be editable.

PR:

#3299
@github-project-automation github-project-automation bot moved this to Backlog in Rafiki Jan 30, 2025
@mkurapov mkurapov mentioned this issue Jan 30, 2025
Open
41 tasks
@mkurapov
Copy link
Contributor Author

mkurapov commented Feb 5, 2025

@koekiebox koekiebox self-assigned this Feb 11, 2025
@koekiebox koekiebox moved this from Backlog to In Progress in Rafiki Feb 11, 2025
@koekiebox koekiebox added the type: enhancement New feature or request label Feb 13, 2025
@koekiebox
Copy link
Collaborator

View for an active tenant:

Image

View only for a deleted tenant:
Image

@koekiebox koekiebox moved this from In Progress to Ready for Review in Rafiki Feb 13, 2025
@koekiebox koekiebox mentioned this issue Feb 13, 2025
Merged
6 tasks
@mkurapov mkurapov linked a pull request Feb 13, 2025 that will close this issue
feat(backend/frontend): 3256 able to view deleted tenant #3299
Merged
6 tasks
@koekiebox koekiebox moved this from Ready for Review to Done in Rafiki Feb 14, 2025
@koekiebox koekiebox closed this as completed by moving to Done in Rafiki Feb 14, 2025
@mkurapov mkurapov mentioned this issue Feb 17, 2025
Merged
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@koekiebox koekiebox

Labels
type: enhancement New feature or request
Projects
Rafiki
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat(backend/frontend): 3256 able to view deleted tenant interledger/rafiki
2 participants
@koekiebox @mkurapov