← 返回首页
Feature/Implement organization quotas version 3 api (issue-1168) by SaifuddinMerchant · Pull Request #1304 · cloudfoundry/cf-java-client · GitHub
Skip to content

Navigation Menu

Toggle navigation
Sign in
Appearance settings
Search or jump to...

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Include my email address so I can be contacted
Cancel Submit feedback

Saved searches

Use saved searches to filter your results more quickly

Cancel Create saved search
Appearance settings
Resetting focus

Feature/Implement organization quotas version 3 api (issue-1168)#1304

Merged
Kehrlann merged 13 commits into
cloudfoundry:mainfrom
SaifuddinMerchant:feature/implement-organizationquotas-version-3-api
Oct 23, 2025
Merged

Feature/Implement organization quotas version 3 api (issue-1168)#1304
Kehrlann merged 13 commits into
cloudfoundry:mainfrom
SaifuddinMerchant:feature/implement-organizationquotas-version-3-api

Conversation

SaifuddinMerchant commented Aug 25, 2025

Copy link
Copy Markdown
Contributor

This provides an implementation of all API for the Organization Quota except for Apply an organization quota to an organization

I plan to implement that Apply API in a future PR. The Organization Quota follows the pattern that Organization version 3 API interface and implementation follow

This PR partially sorts the issue raised for missing version 3 Organization Quota implementation - #1168

issue-1168 adding interface and model for CreateOrganizationQuotaDefi… …
f5d685f
…nition
issue-1168 adding interface and model for GetOrganizationQuotaDefinition
ec7e9b1
issue-1168 adding interface and model for ListOrganizationQuotaDefini… …
ab8bf51
…tion
issue-1168 adding interface and model for UpdateOrganizationQuotaDefi… …
6a5e675
…nition
issue-1168 adding interface and model for DeleteOrganizationQuotaDefi… …
8249d70
…nition
issue-1168 adding reactor implementation for Create, Get, List, Updat… …
e6570c2
…e and Delete OrganizationQuotaDefinition
Merge branch 'main' into feature/implement-organizationquotas-version… …
1e85cb7
…-3-api
issue-1168 adding integration test for version 3 implementation of Or… …
c429295
…ganizationQuotaDefinition
issue-1168 running spotless:apply to fix formatting issues
a1e23cb
issue-1168 running spotless:apply to fix formatting issues
2d2230f
SaifuddinMerchant changed the title Feature/implement organization quotas version 3 api Feature/implement organization quotas version 3 api (issue-1168) Sep 2, 2025
SaifuddinMerchant changed the title Feature/implement organization quotas version 3 api (issue-1168) Feature/Implement organization quotas version 3 api (issue-1168) Sep 2, 2025
issue-1168 added missing Domains property for …
dfc3e9c
_UpdateOrganizationQuotaDefinitionRequest
issue-1168 integration test to use nameFactory.getQuotaDefinitionName… …
4dea9a3
…() so the automated clean-up process can apply
SaifuddinMerchant mentioned this pull request Sep 16, 2025
Closed
Kehrlann self-assigned this Oct 17, 2025
Kehrlann self-requested a review October 17, 2025 12:50

Kehrlann commented Oct 22, 2025

Copy link
Copy Markdown
Contributor

Hey @SaifuddinMerchant ! Sorry for the review delay.

This looks good. Could you please align the "create" integration test with the v2 version? Passing properties and verifying the output.

SaifuddinMerchant commented Oct 22, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Hello @Kehrlann
Integration test has been updated, its slightly different from the v2, since I don't know the Id and created Date - those are auto assigned. I used assert-Next instead of expect-Next.

Kehrlann approved these changes Oct 23, 2025
View reviewed changes

Kehrlann left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality Hide comment

Thank you for your contribution!

Hide details View details Kehrlann merged commit df9ece3 into cloudfoundry:main Oct 23, 2025
5 checks passed
This file contains hidden or 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
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Kehrlann Kehrlann approved these changes

Assignees

Kehrlann

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

Footer

© 2026 GitHub, Inc.