Skip to content

bugfix: make Save button work for new and changed location#1578

Merged
trigal2012 merged 1 commit intodevfrom
location-fix
Feb 1, 2026
Merged

bugfix: make Save button work for new and changed location#1578
trigal2012 merged 1 commit intodevfrom
location-fix

Conversation

@trigal2012
Copy link
Contributor

Pull Request type

Please check the type of change your PR introduces:

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no API changes)
  • Build-related changes
  • Documentation content changes
  • Other (please describe):

What is the current behavior?

Issue Number: N/A

What is the new behavior?

Does this introduce a breaking change?

  • Yes
  • No

Other information

@trigal2012 trigal2012 added the bugfix Inconsistencies or issues which will cause a problem for users or implementors. label Feb 1, 2026
@vercel
Copy link

vercel bot commented Feb 1, 2026

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

Project Deployment Actions Updated (UTC)
inreach-app Ready Ready Preview, Comment Feb 1, 2026 6:29am

Request Review

@github-actions github-actions bot added automerge Enable Kodiak auto-merge kodiak: merge.method = 'squash' Kodiak will squash merge this PR. 📦 ui 📦 api labels Feb 1, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 1, 2026

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (2)
  • packages/api/cache/slugRedirect.ts is excluded by none and included by none
  • packages/ui/components/data-portal/LocationDrawer/index.tsx is excluded by none and included by none

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

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

  • 🔍 Trigger a full review
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch location-fix

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@relativeci
Copy link

relativeci bot commented Feb 1, 2026

#2996 Bundle Size — 5.56MiB (~+0.01%).

4c261cb(current) vs ba94617 dev#2994(baseline)

Warning

Bundle contains 5 duplicate packages – View duplicate packages

Bundle metrics  Change 2 changes Regression 1 regression
                 Current
#2996
     Baseline
#2994
Regression  Initial JS 3.05MiB(~+0.01%) 3.05MiB
No change  Initial CSS 9.7KiB 9.7KiB
Change  Cache Invalidation 22.15% 21.55%
No change  Chunks 61 61
No change  Assets 74 74
No change  Modules 1913 1913
No change  Duplicate Modules 227 227
No change  Duplicate Code 7.45% 7.45%
No change  Packages 158 158
No change  Duplicate Packages 5 5
Bundle size by type  Change 1 change Regression 1 regression
                 Current
#2996
     Baseline
#2994
Regression  JS 4.33MiB (~+0.01%) 4.33MiB
No change  Other 1.12MiB 1.12MiB
No change  Fonts 95.78KiB 95.78KiB
No change  CSS 9.7KiB 9.7KiB
No change  IMG 8.57KiB 8.57KiB

Bundle analysis reportBranch location-fixProject dashboard


Generated by RelativeCIDocumentationReport issue

@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 1, 2026

@alwaysmeticulous
Copy link

🤖 No test run has been triggered as your Meticulous project has been deactivated (since you haven't viewed any test results in a while). Click here to reactivate.

Last updated for commit 4c261cb. This comment will update as new commits are pushed.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 1, 2026

📦 Next.js Bundle Analysis for @weareinreach/app

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

@trigal2012 trigal2012 merged commit b4a181f into dev Feb 1, 2026
26 of 28 checks passed
@trigal2012 trigal2012 deleted the location-fix branch February 1, 2026 06:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

📦 api automerge Enable Kodiak auto-merge bugfix Inconsistencies or issues which will cause a problem for users or implementors. kodiak: merge.method = 'squash' Kodiak will squash merge this PR. 📦 ui

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant