Skip to content

fix: CodeRabbit auto-fixes for PR #22 - #23

Merged
Arun-kushwaha007 merged 3 commits into
issue-bundle-fixesfrom
coderabbitai/autofix/f161b18
Apr 7, 2026
Merged

fix: CodeRabbit auto-fixes for PR #22#23
Arun-kushwaha007 merged 3 commits into
issue-bundle-fixesfrom
coderabbitai/autofix/f161b18

Conversation

@coderabbitai

@coderabbitai coderabbitai Bot commented Apr 7, 2026

Copy link
Copy Markdown
Contributor

This stacked PR contains CodeRabbit auto-fixes for #22.

Files modified:

  • apps/backend/src/activity/activity.service.ts
  • apps/mobile/.turbo/turbo-build.log
  • apps/web/.turbo/turbo-build.log
  • apps/web/src/components/groups/RecurringExpenseList.tsx
  • infra/terraform/.turbo/turbo-build.log
  • packages/shared-types/.turbo/turbo-build.log
  • packages/shared-types/src/index.ts
  • packages/shared-types/tsconfig.json
  • tsconfig.base.json

Fixed 9 file(s) based on 3 unresolved review comments.

Co-authored-by: CodeRabbit <noreply@coderabbit.ai>
@vercel

vercel Bot commented Apr 7, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
fair-share-web Error Error Apr 7, 2026 7:36pm

@coderabbitai

coderabbitai Bot commented Apr 7, 2026

Copy link
Copy Markdown
Contributor Author

Important

Review skipped

This PR was authored by the user configured for CodeRabbit reviews. CodeRabbit does not review PRs authored by this user. It's recommended to use a dedicated user account to post CodeRabbit review feedback.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: cadf62f2-32c7-4400-a5cf-8fc262cbdd9c

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Comment @coderabbitai help to get the list of available commands and usage tips.

@Arun-kushwaha007
Arun-kushwaha007 merged commit 178e243 into issue-bundle-fixes Apr 7, 2026
1 of 2 checks passed
Arun-kushwaha007 added a commit that referenced this pull request Apr 7, 2026
…cy formatting, and web auth (#22)

* Fixes #2, Fixes #3, Fixes #5, Fixes #20: unify activity names, currency formatting, and web auth

* 📝 Add docstrings to `issue-bundle-fixes`

Docstrings generation was requested by @Arun-kushwaha007.

The following files were modified:

* `apps/mobile/app/components/ui/MoneyText.tsx`
* `apps/mobile/app/screens/ActivityScreen.tsx`
* `apps/mobile/app/screens/AddExpenseScreen.tsx`
* `apps/mobile/app/screens/ExpenseDetailScreen.tsx`
* `apps/mobile/app/screens/GroupDetailScreen.tsx`
* `apps/mobile/app/screens/GuestGroupDetailScreen.tsx`
* `apps/mobile/app/screens/HomeScreen.tsx`
* `apps/web/app/dashboard/activity/page.tsx`
* `apps/web/app/dashboard/groups/[groupId]/page.tsx`
* `apps/web/app/dashboard/page.tsx`
* `apps/web/app/share/[token]/page.tsx`
* `apps/web/src/components/activity/ActivityFeed.tsx`
* `apps/web/src/components/dashboard/ActivityList.tsx`
* `apps/web/src/components/groups/ExpenseDetailCard.tsx`
* `apps/web/src/components/groups/ExpenseRow.tsx`
* `apps/web/src/components/groups/ExpenseTable.tsx`
* `apps/web/src/components/groups/GroupSummaryPanel.tsx`
* `apps/web/src/components/groups/RecurringExpenseList.tsx`
* `apps/web/src/components/groups/SettlementList.tsx`
* `packages/shared-types/src/index.ts`

These files were ignored:
* `apps/backend/src/activity/activity.service.spec.ts`

These file types are not supported:
* `README.md`

* fix: improve type safety and input validation

- Update activity type casting in backend service for better Prisma compatibility
- Refine currency prop type in web component to prevent type mismatches
- Add accessibility titles to select elements in recurring expense list
- Enhance currency cents input normalization with integer validation in shared types

Fixes #2, #3, #5, #20

* fix: apply CodeRabbit auto-fixes

Fixed 7 file(s) based on 1 unresolved review comment.

Co-authored-by: CodeRabbit <noreply@coderabbit.ai>

* fix: apply CodeRabbit auto-fixes (#23)

Fixed 9 file(s) based on 3 unresolved review comments.

Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Co-authored-by: CodeRabbit <noreply@coderabbit.ai>
Co-authored-by: Arun Kushwaha <121769639+Arun-kushwaha007@users.noreply.github.com>

* 📝 Add docstrings to `issue-bundle-fixes`

Docstrings generation was requested by @Arun-kushwaha007.

The following files were modified:

* `apps/mobile/app/components/ui/MoneyText.tsx`
* `apps/mobile/app/screens/ActivityScreen.tsx`
* `apps/mobile/app/screens/ExpenseDetailScreen.tsx`
* `apps/mobile/app/screens/GuestGroupDetailScreen.tsx`
* `apps/mobile/app/screens/HomeScreen.tsx`
* `apps/web/app/dashboard/groups/[groupId]/page.tsx`
* `apps/web/app/dashboard/page.tsx`
* `apps/web/src/components/activity/ActivityFeed.tsx`
* `apps/web/src/components/dashboard/ActivityList.tsx`
* `apps/web/src/components/groups/GroupSummaryPanel.tsx`
* `apps/web/src/components/groups/RecurringExpenseList.tsx`
* `apps/web/src/components/groups/SettlementList.tsx`
* `packages/shared-types/src/index.ts`

These files were kept as they were:
* `apps/mobile/app/screens/AddExpenseScreen.tsx`
* `apps/mobile/app/screens/GroupDetailScreen.tsx`
* `apps/web/app/dashboard/activity/page.tsx`
* `apps/web/app/share/[token]/page.tsx`
* `apps/web/src/components/groups/ExpenseDetailCard.tsx`
* `apps/web/src/components/groups/ExpenseRow.tsx`
* `apps/web/src/components/groups/ExpenseTable.tsx`

These files were ignored:
* `apps/backend/src/activity/activity.service.spec.ts`

These file types are not supported:
* `README.md`
* `packages/shared-types/tsconfig.json`
* `tsconfig.base.json`

* Fix GitHub Actions build failures and configure gitignore

---------

Co-authored-by: root <root@localhost.localdomain>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Co-authored-by: CodeRabbit <noreply@coderabbit.ai>
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.

1 participant