Thanks to visit codestin.com
Credit goes to github.com

Skip to content

improvement of constants: unify ChatLimit #3377

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.

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

Merged
merged 1 commit into from
Nov 22, 2022

Conversation

lemontree210
Copy link
Member

@lemontree210 lemontree210 commented Nov 22, 2022

This is a tiny PR to follow up on this comment in #3351 pertaining to issue #3107.

The only class I found that could be merged into ChatLimit, considering the comment mentioned above, was ChatDescriptionLimit.

@lemontree210 lemontree210 added the ⚙️ documentation affected functionality: documentation label Nov 22, 2022
@lemontree210 lemontree210 self-assigned this Nov 22, 2022
@Bibo-Joshi Bibo-Joshi added 🛠 refactor change type: refactor and removed ⚙️ documentation affected functionality: documentation labels Nov 22, 2022
@Bibo-Joshi Bibo-Joshi merged commit 6cba7f1 into master Nov 22, 2022
@Bibo-Joshi Bibo-Joshi deleted the unified-chat-limit-in-constants branch November 22, 2022 11:09
@harshil21 harshil21 added this to the v20.0a5 milestone Nov 24, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Dec 1, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🛠 refactor change type: refactor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants