Skip to content

Mobile tasks fresh branch#293

Open
spokonya wants to merge 10 commits intomainfrom
mobile-tasks-freshBranch
Open

Mobile tasks fresh branch#293
spokonya wants to merge 10 commits intomainfrom
mobile-tasks-freshBranch

Conversation

@spokonya
Copy link
Copy Markdown
Contributor

@spokonya spokonya commented Apr 7, 2026

Description

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Refactoring (code improvement without changing functionality)
  • Documentation update
  • Configuration/infrastructure change
  • Performance improvement
  • Test coverage improvement

Related Issue(s)

Closes #
Related to #126

What Changed?

-mobile Tasks page with My Tasks/Unassigned tabs, server-side filters, client-side search, task detail and filter bottom sheets, and a completion popup
-Backend task endpoints (GET, POST, PATCH, claim, drop)
-Removed all hardcoded auth bypasses (X-Dev-User-Id, X-Hotel-ID, dev Clerk user ID config) and added migrations

Testing & Validation

How this was tested

Screenshots/Recordings

ScreenRecording_04-07-2026.13-00-15_1.MP4
image Screenshot 2026-04-07 at 3 48 52 PM Screenshot 2026-04-07 at 3 48 35 PM

Unfinished Work & Known Issues

  • None, this PR is complete and production-ready
  • The following items are intentionally deferred:
    • creating new task
      -Hi-fi

Notes & Nuances



Pre-Merge Checklist

Code Quality

  • Code follows the project's style guidelines and conventions
  • Self-review completed (I've reviewed my own code for obvious issues)
  • No debugging code, console logs, or commented-out code left behind
  • No merge conflicts with the base branch
  • Meaningful commit messages that explain the "why"

Testing & CI

  • All CI checks are passing
  • All new and existing tests pass locally
  • Test coverage hasn't decreased (or decrease is justified)
  • Linting passes without errors

Documentation

  • Code is self-documenting or includes helpful comments for complex logic
  • API documentation updated (if backend endpoints changed)
  • Type definitions are accurate and up-to-date

Reviewer Notes

  • Areas needing extra attention: ...
  • Questions for reviewers: ...

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 7, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 5.54%. Comparing base (6fc7227) to head (4d1760c).

Additional details and impacted files

Impacted file tree graph

@@          Coverage Diff          @@
##            main    #293   +/-   ##
=====================================
  Coverage   5.54%   5.54%           
=====================================
  Files         94      94           
  Lines       4476    4476           
  Branches      21      21           
=====================================
  Hits         248     248           
  Misses      4228    4228           
Flag Coverage Δ
mobile 97.29% <ø> (ø)
web 4.77% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 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.

@danctila danctila added the feature Introduces a new and complete feature label Apr 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature Introduces a new and complete feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants