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

Satellite admin web App: View project info #6476

Closed
ifraixedes opened this issue Nov 2, 2023 · 9 comments
Closed

Satellite admin web App: View project info #6476

ifraixedes opened this issue Nov 2, 2023 · 9 comments
Assignees
@ifraixedes
Labels
Enhancement New feature or request

Comments

@ifraixedes
Copy link
Member

ifraixedes commented Nov 2, 2023 •
edited

Motivation / Why?

The new satellite administration Web App shows project information.

What needs to be done?

  • Define API endpoint to show project information through the API generator.
  • Implement the business logic of the endpoint.
  • Wire the front-end with the API client generated by the API generator.
  • Implement the needed front-end logic to accommodate the API client call.

Out of scope:

  • To show project stats: number of segments, used storage, used bandwidth, etc.

Acceptance criteria

  • The new web App allows to click on the list of user's projects to navigate to a project information page.
  • The new web App shows project information.
@ifraixedes ifraixedes added the Enhancement New feature or request label Nov 2, 2023
@ifraixedes ifraixedes self-assigned this Nov 28, 2023
@ifraixedes
Copy link
Member Author

Blocked by #6581

@ifraixedes ifraixedes added the blocked For tagging issues that are blocked label Dec 5, 2023
@storj-gerrit
Copy link

storj-gerrit bot commented Dec 8, 2023

Change satellite/admin: define project info endpoint mentions this issue.

@storj-gerrit
Copy link

storj-gerrit bot commented Dec 8, 2023 •
edited by ifraixedes

Change satellite/admin: WIP implement back-office project view mentions this issue.

@ifraixedes ifraixedes removed the blocked For tagging issues that are blocked label Dec 14, 2023
@ifraixedes
Copy link
Member Author

For now, we are skipping the embedded structs without JSON tag, so I unblocked it

@ifraixedes
Copy link
Member Author

It turns out that I implemented #6581 and I used it.

Commit ready for reviews https://review.dev.storj.io/c/storj/storj/+/11927

@ifraixedes
Copy link
Member Author

ifraixedes commented Dec 20, 2023 •
edited

The UI part is missing, I will post the commit link once it's done, however, the API part is ready, as commented just above.

@storj-gerrit
Copy link

storj-gerrit bot commented Dec 20, 2023

Change satellite/admin: WIP implement back-office project view mentions this issue.

storjBuildBot pushed a commit that referenced this issue Jan 4, 2024
@ifraixedes @cam-a
satellite/admin: define project info endpoint
0bdb4f9 
Implement API endpoint to get project info.

Additionally, rename admin back-office User to UserAccount and split
fields ID, FullName, and Email into new User struct for reuse in Project
struct.

Move the logic of getting project limits out from the GetUser service
method to be able to reuse it between it and the new GetProject servie
method.

issue: #6476

Change-Id: I5d381359c2bf2616843c3c6a8a5f1ade09cbf867
storjBuildBot pushed a commit that referenced this issue Jan 8, 2024
@cam-a @ifraixedes
satellite/admin/back-office/ui: Project view
63e67cf 
This commit map the get project API endpoint to the UI to show its
information in the project view.

issue: #6476

Change-Id: I50d065b65c2de2425d7e90720b18a3ea00fc7fcf
@ifraixedes
Copy link
Member Author

Thanks to @cam-a the project view UI is implemented and merged.

@ifraixedes
Copy link
Member Author

Deployed in release v1.96.6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ifraixedes ifraixedes

Labels
Enhancement New feature or request
Projects
Team Delivery
Status: Done/Deployed
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ifraixedes