Skip to content

Conversation

@Tisho99
Copy link
Contributor

@Tisho99 Tisho99 commented Dec 12, 2025

T-8768

@codecov
Copy link

codecov bot commented Dec 12, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

Files with missing lines Coverage Δ
duplicate_project_task_timesheet/tests/__init__.py 100.00% <100.00%> (ø)
...eet/tests/test_duplicate_project_task_timesheet.py 100.00% <100.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link

@javierizaca javierizaca left a comment

Choose a reason for hiding this comment

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

Hi @Tisho99 !

The functional review is correct; project task timesheet lines can be easily duplicated.

It would be necessary to add the task to the PR title [T-8768] for it to be correct.

I believe this is happening in several PRs for the same task. Could you please review them?

@Tisho99 Tisho99 changed the title [18.0] [MIG] duplicate_project_task_timesheet [18.0][T-8768][MIG] duplicate_project_task_timesheet Jan 19, 2026
Copy link

@javierizaca javierizaca left a comment

Choose a reason for hiding this comment

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

LGTM

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants