Skip to content

[server] Hook up usage API (4/5) #11322

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
Merged
roboquat merged 1 commit into from
Jul 13, 2022
Merged

[server] Hook up usage API (4/5) #11322

roboquat merged 1 commit into from
Jul 13, 2022

Conversation

@geropl
Copy link
Member

@geropl geropl commented Jul 12, 2022 •
edited
Loading

Description

Configures server to consume the actual UsageService over gRPC.

Related Issue(s)

Context #10325

How to test

  • create a team with billing
  • go to team -> Billed Usage and see that empty list we currently return as dummy data (and notice that there are no errors in the web console)

Release Notes

NONE

Documentation

Werft options:

  • /werft with-preview
  • /werft with-payment

@geropl geropl force-pushed the gpl/10325-useapi-4 branch from 2a2eb83 to d09b778 Compare July 13, 2022 06:36
@roboquat roboquat added size/L and removed size/XXL labels Jul 13, 2022
@geropl geropl force-pushed the gpl/10325-useapi-4 branch 2 times, most recently from b42e693 to d6d8a76 Compare July 13, 2022 09:04
@geropl geropl marked this pull request as ready for review July 13, 2022 09:16
@geropl geropl requested a review from a team July 13, 2022 09:16
@github-actions github-actions bot added the team: webapp Issue belongs to the WebApp team label Jul 13, 2022
geropl
geropl commented Jul 13, 2022
View reviewed changes
easyCZ
easyCZ approved these changes Jul 13, 2022
View reviewed changes
Copy link
Member

@easyCZ easyCZ left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/hold
for comments, up to you to make changes or not

@geropl geropl force-pushed the gpl/10325-useapi-4 branch from d6d8a76 to 7e21e49 Compare July 13, 2022 09:31
easyCZ
easyCZ reviewed Jul 13, 2022
View reviewed changes
@geropl geropl force-pushed the gpl/10325-useapi-4 branch from 7e21e49 to eda6872 Compare July 13, 2022 09:59
@geropl
Copy link
Member Author

geropl commented Jul 13, 2022

/unhold

Now with dummydata again:
image

@roboquat roboquat merged commit d1c412e into main Jul 13, 2022
@roboquat roboquat deleted the gpl/10325-useapi-4 branch July 13, 2022 10:09
@roboquat roboquat added deployed: webapp Meta team change is running in production deployed Change is completely running in production labels Jul 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@easyCZ easyCZ easyCZ approved these changes

Assignees

No one assigned

Labels

deployed: webapp Meta team change is running in production deployed Change is completely running in production release-note-none size/L team: webapp Issue belongs to the WebApp team

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@geropl @easyCZ @roboquat