chore(mono): 🚧 Update frontend/v2 with base changes of main branch#49
Closed
Nudelsuppe42 wants to merge 51 commits intofrontend/v2from
Closed
chore(mono): 🚧 Update frontend/v2 with base changes of main branch#49Nudelsuppe42 wants to merge 51 commits intofrontend/v2from
Nudelsuppe42 wants to merge 51 commits intofrontend/v2from
Conversation
…llery page. fixes #13
…ar nav, option to hide sidebar nav when not placed in group -> preperation for claim editor
Added mapbox-gl-draw, mapbox-gl-draw-snap-mode and turf. Area editing, creation and deletion is fully working. TODO: Turn on/off snapping -> always on is too resource heavy, redirect to advanced element editor
…ery file for better organization
…aw-ids which are not valid UUIDS for backwards compatilbility with frontend
Also remove debug logs for weird prisma error since its fixed
…themecolor to static blue [skip ci]
... and prettier
bruh prettier
There was a problem hiding this comment.
Pull Request Overview
This PR syncs frontend/v2 with the latest main branch by applying core dependency upgrades, formatting config updates, and widespread UI refactors—most notably renaming “BuildTeam” to “Build Region” and replacing manual <Box> wrappers with a shared ContentWrapper component.
- Dependency updates: bumped Prisma to v6, Turbo to 2.5.4, removed a Prettier plugin.
- Layout refactors: replaced per-page
<Box>margins/widths withContentWrapper, updated many pages and loading states. - Terminology standardization: renamed labels, variables, and comments from “Team” to “Region” throughout the dashboard.
Reviewed Changes
Copilot reviewed 128 out of 144 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| packages/prettier-config/index.json | Removed plugins entry for organize-imports. |
| packages/db/prisma/schema.prisma | Commented out unused binaryTargets generator option. |
| packages/db/prisma/migrations/migration_lock.toml | Grammar tweak in comment. |
| packages/db/prisma/migrations/20250613201530_update_to_prisma_v6/ | Added new migration SQL for composite PK on _builders and _members. |
| packages/db/package.json | Upgraded @prisma/client and prisma to v6; changed migrate script. |
| package.json | Bumped turbo to ^2.5.4. |
| apps/frontend/src/pages/gallery.tsx | Simplified default image hash literal; expanded map callback. |
| apps/frontend/src/components/GalleryGrid.tsx | Removed stray whitespace and a commented-out style. |
| apps/frontend/src/components/BackgroundImage.tsx | Cleaned up props, unified alt text, simplified placeholder logic. |
| apps/frontend/public/locales/en/getstarted.json | Fixed typo: “build” → “built.” |
| apps/dashboard/src/util/webhooks.ts | Added sendBtWebhook helper to post JSON webhooks. |
| apps/dashboard/src/util/transformers.ts | Renamed tooltip/error text references from “team” to “region.” |
| apps/dashboard/src/components/layout/** | Replaced <Box> with ContentWrapper, added ScrollArea, updated nav. |
| apps/dashboard/src/components/core/card/TextCard.tsx | Added optional href button, external-link icon. |
| ...many more across dashboard app layout, hooks, inputs, and APIs | Standardized styling, responsiveness, and component signatures. |
Comments suppressed due to low confidence (2)
apps/dashboard/src/components/core/card/TextCard.tsx:32
- There is an extra closing
</Group>here, which likely mis-nests the component tree and breaks the intended layout. Please remove the redundant closing tag.
>
packages/db/prisma/schema.prisma:3
- [nitpick] Consider removing or moving this commented-out
binaryTargetsline into documentation if it’s no longer needed, to keep the schema file clean.
// binaryTargets = ["native", "linux-musl","linux-musl-openssl-3.0.x"]
| return false; | ||
| } | ||
| } | ||
| } |
There was a problem hiding this comment.
The function sendBtWebhook returns false on error but does not return anything on success, leading to an inconsistent return type. Consider returning true after a successful fetch.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.