8000 Don't send Assigned Ticket Response Time Alert for Gumroad during weekends by devin-ai-integration[bot] · Pull Request #14 · antiwork/helper · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Don't send Assigned Ticket Response Time Alert for Gumroad during weekends #14

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 4 commits into from
Mar 3, 2025

Conversation

devin-ai-integration[bot]
Copy link
Contributor

This PR modifies the Assigned Ticket Response Time Alert to not send during weekends for Gumroad, as requested in the Slack channel #gumroad-support.

Link to Devin run: https://app.devin.ai/sessions/5cd5fb95455342aca93adb8b279f4cc9

…kends

Co-Authored-By: sahil.lavingia@gmail.com <sahil.lavingia@gmail.com>
Copy link
Contributor Author

🤖 Devin AI Engineer

Original prompt from sahil.lavingia@gmail.com:

SYSTEM:
The following is the thread conversation in Slack channel #gumroad-support that you've been added to:

=== BEGIN THREAD HISTORY ===
U06CARC3MKM: Assigned Ticket Response Time Alert for Gumroad



U024FUD6D: @Devin in anti-work/helper don’t send this alert during the weekend 


=== END THREAD HISTORY ===

The latest message is the one right above that tagged you.

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add "(aside)" to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

Copy link
supabase bot commented Mar 3, 2025

This pull request has been ignored for the connected project vpcvnictyvrybuwjrjzn because there are no changes detected in supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

});
// Don't send alerts during weekends for Gumroad
const now = new Date();
if (!isWeekend(now) || mailbox.name !== "Gumroad") {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For all mailboxes not just gumroad

Co-Authored-By: sahil.lavingia@gmail.com <sahil.lavingia@gmail.com>
});
// Don't send alerts during weekends
const now = new Date();
if (!isWeekend(now)) {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do this check at the top of the function so it doesn't run the check at all if it's a weekend

Co-Authored-By: sahil.lavingia@gmail.com <sahil.lavingia@gmail.com>
Signed-off-by: Jono M <reason.koan@gmail.com>
@binary-koan binary-koan merged commit 229460b into main Mar 3, 2025
5 of 7 checks passed
@binary-koan binary-koan deleted the devin/1740971180-disable-weekend-alert branch March 3, 2025 22:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0