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

[🐛 Bug]: Merge incidents not working in some cases #3370

Closed
talboren opened this issue Feb 10, 2025 · 4 comments · Fixed by #4058
Closed

[🐛 Bug]: Merge incidents not working in some cases #3370

talboren opened this issue Feb 10, 2025 · 4 comments · Fixed by #4058
Assignees
@Matvey-Kuk
Labels
💎 Bounty Bug Something isn't working UI User interface related issues

Comments

@talboren
Copy link
Member

While the UI shows success:

Image

The actual response from the backend is:

Image
@dosubot dosubot bot added Bug Something isn't working UI User interface related issues labels Feb 10, 2025
@Matvey-Kuk
Copy link
Contributor

/bounty 20

@algora-pbc Algora PBC
Copy link

algora-pbc bot commented Feb 14, 2025 •
edited
Loading

💎 $20 bounty • Keep (YC W23)

Steps to solve:

  1. Start working: Comment /attempt #3370 with your implementation plan
  2. Submit work: Create a pull request including /claim #3370 in the PR body to claim the bounty
  3. Receive payment: 100% of the bounty is received 2-5 days post-reward. Make sure you are eligible for payouts

Thank you for contributing to keephq/keep!

Add a bounty • Share on socials

Attempt Started (GMT+0) Solution
🟢 @AkashJana18 Feb 14, 2025, 6:22:29 AM WIP
🔴 @nayan-bagale Feb 14, 2025, 6:25:39 PM WIP

@AkashJana18
Copy link

AkashJana18 commented Feb 14, 2025 •
edited by algora-pbc bot
Loading

/attempt #3370

Hey @Matvey-Kuk I would like to work on this issue. Can you please assign it to me. Thank you

Options

@nayan-bagale nayan-bagale mentioned this issue Feb 14, 2025
Closed
4 tasks
@nayan-bagale
Copy link

nayan-bagale commented Feb 14, 2025 •
edited by algora-pbc bot
Loading

/attempt #3370

Hey @Matvey-Kuk, I’ve created a PR. I replaced toast.success with toast.info and passed result.message as an argument since the response contains a different message. A success toast isn’t necessary in this case

Options

This was referenced Feb 14, 2025
Closed
Closed
@dosubot dosubot bot mentioned this issue Feb 27, 2025
Open
@Matvey-Kuk Matvey-Kuk self-assigned this Mar 5, 2025
@dosubot dosubot bot mentioned this issue Mar 16, 2025
Open
@Matvey-Kuk Matvey-Kuk mentioned this issue Mar 16, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Matvey-Kuk Matvey-Kuk

Labels
💎 Bounty Bug Something isn't working UI User interface related issues
Projects
None yet
Milestone
No milestone
4 participants
@Matvey-Kuk @nayan-bagale @talboren @AkashJana18