-
-
Notifications
You must be signed in to change notification settings - Fork 476
feat: added Interaction.attachment_size_limit #2854
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
Conversation
Signed-off-by: hich4t <[email protected]>
Signed-off-by: hich4t <[email protected]>
Signed-off-by: hich4t <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, thanks for the pr ! Could you please do the following to make sure the pr is merged:
- Edit the
CHANGELOG.md
file at the root of the repository to add under the## Added
section your addition, making sure to keep the same format as the other entries. - Add a shortcut as a property to
ApplicationContext
namedattachment_size_limit
that returns the related interaction'sattachment_size_limit
. - Add
attachment_size_limit
to theInteraction
's type class.
Signed-off-by: hich4t <[email protected]>
Signed-off-by: hich4t <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you missed the third one
Add
attachment_size_limit
to theInteraction
's type class.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm
Oops I missed it bc the two commits were named the same |
Co-authored-by: Paillat <[email protected]> Signed-off-by: hich4t <[email protected]>
Co-authored-by: Paillat <[email protected]> Signed-off-by: hich4t <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Summary
added Interaction.attachment_size_limit
Information
examples, ...).
Checklist
type: ignore
comments were used, a comment is also left explaining why.