Skip to content
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

fix: trim github notification message - RK-19530 #113

Merged
merged 1 commit into from
Jul 27, 2023
Merged

fix: trim github notification message - RK-19530 #113

merged 1 commit into from
Jul 27, 2023

Conversation

gosharo
Copy link
Contributor

@gosharo gosharo commented Jul 26, 2023 •
edited
Loading

Signed-off-by: Gosha [email protected]

Pull Request

Description

Github notification gets 422 error because the message too long.
Screenshot 2023-07-26 at 19 10 20

Checklist

Before submitting this pull request, please ensure that you have completed the following tasks:

@gosharo gosharo requested a review from nissim-infra as a code owner July 26, 2023 16:09
@sonariorobot sonariorobot changed the title fix: trim github notifiction massge fix: trim github notifiction massge - RK-19529 Jul 26, 2023
@sonariorobot sonariorobot changed the title fix: trim github notifiction massge - RK-19529 fix: trim github notifiction massge - RK-19530 Jul 26, 2023
@gosharo gosharo added the DeployEnforcer/MergeOnceApproved Auto merge and clean branch once passed all checks label Jul 26, 2023
@ElDuderinos ElDuderinos changed the title fix: trim github notifiction massge - RK-19530 fix: trim github notification message - RK-19530 Jul 27, 2023
@gosharo gosharo merged commit 75464e3 into main Jul 27, 2023
@gosharo gosharo deleted the trim-notification-message-length branch July 27, 2023 12:18
@gosharo gosharo mentioned this pull request Jul 27, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ElDuderinos ElDuderinos ElDuderinos approved these changes

@nissim-infra nissim-infra Awaiting requested review from nissim-infra

@EliRookout EliRookout Awaiting requested review from EliRookout

Assignees
No one assigned
Labels
DeployEnforcer/MergeOnceApproved Auto merge and clean branch once passed all checks
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@gosharo @ElDuderinos