[db][protocol] Implement a CostCenter entity to attribute workspace usage to #10788
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
@geropl Apologies for rushing into this 🙈 I actually implemented all this before I saw your request for further alignment, which I agree with: #10757 (comment) So, I'm happy to further align on this concept and on naming, then reflect any changes back in this PR. 👍 |
jankeromnes
force-pushed
the
jx/cost-center
branch
from
d7ea0e4
to
df56997
Compare
jankeromnes
commented
Jun 24, 2022
components/gitpod-db/src/typeorm/migration/1655795038249-CostCenter.ts
Outdated
Show resolved
Hide resolved
jankeromnes
commented
Jun 24, 2022
components/gitpod-db/src/typeorm/migration/1655795038249-CostCenter.ts
Outdated
Show resolved
Hide resolved
jankeromnes
commented
Jun 24, 2022
components/gitpod-db/src/typeorm/migration/1655795038249-CostCenter.ts
Outdated
Show resolved
Hide resolved
|
Whoops, looks like this suffered from bit-rot a bit. If anyone would like to pick this up, please feel free to force-push to this branch, or salvage anything that looks useful from it (no need to attribute me 😊). |
|
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
AlexTugarev
force-pushed
the
jx/cost-center
branch
from
df56997
to
ea85038
Compare
easyCZ
reviewed
Jul 18, 2022
AlexTugarev
force-pushed
the
jx/cost-center
branch
from
ea85038
to
54914fb
Compare
AlexTugarev
force-pushed
the
jx/cost-center
branch
from
cc7e6e7
to
d65afc6
Compare
|
How is no-spending limit represented here? |
roboquat
added
deployed: webapp
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Implement a CostCenter entity to attribute workspace usage to.
Related Issue(s)
Fixes #10757
How to test
Release Notes
Documentation
Werft options: