Skip to content

Navigation Menu

Sign in
Appearance settings

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

Provide feedback

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

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

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.

4 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.