Skip to content

Conversation

@ariedov
Copy link
Contributor

@ariedov ariedov commented Dec 12, 2025

Description

Bugfix release, the bug was caught by the previous auto-review when updating the task priority resulted in 403 on the Tududi backend. Using uid there instead of id.

Checklist

- fix: using uid instead of id to update task priority
- Pull contributions
@raycastbot raycastbot added extension fix / improvement Label for PRs with extension's fix improvements extension: tududi Issues related to the tududi extension platform: macOS platform: Windows OP is author The OP of the PR is the author of the extension labels Dec 12, 2025
@raycastbot
Copy link
Collaborator

Thank you for the update! 🎉

You can expect an initial review within five business days.

@ariedov ariedov marked this pull request as ready for review December 12, 2025 10:13
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Dec 12, 2025

Greptile Overview

Greptile Summary

This PR fixes a bug where updating task priority in the "Today Tasks" view resulted in a 403 error from the Tududi backend. The fix changes line 136 in today-tasks.tsx to use task.uid instead of task.id when making the PATCH request to /api/task/{id}.

  • Fixed updateTaskPriority function to use task.uid in the API endpoint, consistent with updateTaskStatus and toggleToday functions
  • Updated CHANGELOG.md with version 1.8.1 entry documenting the bugfix
  • This aligns with the technical improvement made in version 1.8.0 which standardized on using uid for API calls

Confidence Score: 5/5

  • This PR is safe to merge with no risk
  • The change is a simple one-line bugfix that corrects an inconsistency in the API call parameter. The fix uses task.uid which is already used correctly in all other API calls in the same file (updateTaskStatus and toggleToday functions), making the code consistent across the codebase. The change directly addresses the 403 error issue and aligns with the architectural decision documented in version 1.8.0 to use uid instead of id for API endpoints.
  • No files require special attention

Important Files Changed

File Analysis

Filename Score Overview
extensions/tududi/CHANGELOG.md 5/5 Added version 1.8.1 entry with bugfix description, correctly formatted with {PR_MERGE_DATE} template
extensions/tududi/src/today-tasks.tsx 5/5 Fixed API call to use task.uid instead of task.id in updateTaskPriority function, consistent with other API calls

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

2 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

Copy link
Collaborator

@pernielsentikaer pernielsentikaer left a comment

Choose a reason for hiding this comment

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

Looks good to me, approved 🔥

@raycastbot raycastbot merged commit 1ae4316 into raycast:main Dec 15, 2025
2 checks passed
@github-actions
Copy link
Contributor

Published to the Raycast Store:
https://raycast.com/david_leibovych/tududi

@raycastbot
Copy link
Collaborator

🎉 🎉 🎉

We've rewarded your Raycast account with some credits. You will soon be able to exchange them for some swag.

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

Labels

extension fix / improvement Label for PRs with extension's fix improvements extension: tududi Issues related to the tududi extension OP is author The OP of the PR is the author of the extension platform: macOS platform: Windows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants