Skip to content

Fix limit object usage in sample #1751

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
merged 1 commit into from
Oct 22, 2025
Merged

Fix limit object usage in sample #1751

merged 1 commit into from
Oct 22, 2025

Conversation

@habara-k
Copy link
Contributor

@habara-k habara-k commented Oct 22, 2025

Changes

Refactored Limit initialization to use the builder pattern:

new Limit.Builder().max(maxLimit).upToRemainingQuota(false).build()

This prevents build failures when new fields are added to the Limit class.

@habara-k habara-k mentioned this pull request Oct 22, 2025
Merged
@habara-k habara-k requested a review from a team October 22, 2025 02:27
@habara-k habara-k added this pull request to the merge queue Oct 22, 2025
Merged via the queue into master with commit 7c80ecd Oct 22, 2025
10 checks passed
@habara-k habara-k deleted the fix-limit-object-usage-in-sample branch October 22, 2025 03:09
@habara-k habara-k mentioned this pull request Oct 30, 2025
Merged
github-merge-queue bot pushed a commit that referenced this pull request Oct 30, 2025
@habara-k
Fix message event usage in test (#1761)
3775ca9 
Similar with #1751.

## Changes

Refactored `MessageEvent` initialization to use the builder pattern:

```java
new MessageEvent.Builder(...).build()
```

This prevents build failures when new fields are added to the
`MessageEvent` class.
@habara-k habara-k mentioned this pull request Nov 6, 2025
Merged
github-merge-queue bot pushed a commit that referenced this pull request Nov 6, 2025
@habara-k
Fix the usage of message content in test (#1769)
75673cb 
Similar with #1751 and
#1761

## Changes

Refactored `XxxMessageContent` initialization to use the builder
pattern:

```java
new XxxMessageContent.Builder(...).build()
```

This prevents build failures when new fields are added to the
`XxxMessageContent` class.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mokuzon mokuzon mokuzon approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@habara-k @mokuzon