Skip to content

fix: Tasker new Date is fixed to lambda lifetime#17342

Merged
zomars merged 2 commits intomainfrom
hotfix/tasker-new-date
Oct 25, 2024
Merged

fix: Tasker new Date is fixed to lambda lifetime#17342
zomars merged 2 commits intomainfrom
hotfix/tasker-new-date

Conversation

@zomars
Copy link
Copy Markdown
Contributor

@zomars zomars commented Oct 25, 2024

What does this PR do?

This pull request to packages/features/tasker/repository.ts includes changes to ensure that the whereUpcomingTasks condition always uses the current date. The most important changes involve refactoring the whereUpcomingTasks constant into a function to dynamically generate the query condition.

Refactoring whereUpcomingTasks:

  • Fixes #XXXX (GitHub issue number)
  • Fixes CAL-XXXX (Linear issue number - should be visible at the bottom of the GitHub issue description)

Mandatory Tasks (DO NOT REMOVE)

  • I have self-reviewed the code (A decent size PR without self-review might be rejected).
  • I have updated the developer docs in /docs if this PR makes changes that would require a documentation change. If N/A, write N/A here and check the checkbox.
  • I confirm automated tests are in place that prove my fix is effective or that my feature works.

How should this be tested?

  • Are there environment variables that should be set?
  • What are the minimal test data to have?
  • What is expected (happy path) to have (input and output)?
  • Any other important info that could help to test that PR

Checklist

  • I haven't read the contributing guide
  • My code doesn't follow the style guidelines of this project
  • I haven't commented my code, particularly in hard-to-understand areas
  • I haven't checked if my changes generate no new warnings

Signed-off-by: Omar López <zomars@me.com>
@github-actions
Copy link
Copy Markdown
Contributor

Hey there and thank you for opening this pull request! 👋🏼

We require pull request titles to follow the Conventional Commits specification and it looks like your proposed title needs to be adjusted.

Details:

Unknown release type "hotfix" found in pull request title "hotfix: Tasker new Date is fixed to lambda lifetime". 

Available types:
 - feat: A new feature
 - fix: A bug fix
 - docs: Documentation only changes
 - style: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc)
 - refactor: A code change that neither fixes a bug nor adds a feature
 - perf: A code change that improves performance
 - test: Adding missing tests or correcting existing tests
 - build: Changes that affect the build system or external dependencies (example scopes: gulp, broccoli, npm)
 - ci: Changes to our CI configuration files and scripts (example scopes: Travis, Circle, BrowserStack, SauceLabs)
 - chore: Other changes that don't modify src or test files
 - revert: Reverts a previous commit

@dosubot dosubot bot added the 🐛 bug Something isn't working label Oct 25, 2024
@keithwillcode keithwillcode added core area: core, team members only foundation labels Oct 25, 2024
@graphite-app graphite-app bot requested a review from a team October 25, 2024 19:59
@graphite-app
Copy link
Copy Markdown

graphite-app bot commented Oct 25, 2024

Graphite Automations

"Add foundation team as reviewer" took an action on this PR • (10/25/24)

1 reviewer was added to this PR based on Keith Williams's automation.

"Add ready-for-e2e label" took an action on this PR • (10/25/24)

1 label was added to this PR based on Keith Williams's automation.

CarinaWolli
CarinaWolli previously approved these changes Oct 25, 2024
@vercel
Copy link
Copy Markdown

vercel bot commented Oct 25, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
cal ⬜️ Ignored (Inspect) Visit Preview Oct 25, 2024 8:33pm
calcom-web-canary ⬜️ Ignored (Inspect) Visit Preview Oct 25, 2024 8:33pm

keithwillcode
keithwillcode previously approved these changes Oct 25, 2024
@keithwillcode keithwillcode changed the title hotfix: Tasker new Date is fixed to lambda lifetime fix: Tasker new Date is fixed to lambda lifetime Oct 25, 2024
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Oct 25, 2024

E2E results are ready!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛 bug Something isn't working core area: core, team members only foundation ready-for-e2e

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants