Skip to content

fix: update app description to reflect accurate location#187

Merged
vbuch merged 25 commits intovbuch:mainfrom
bancho22:fix/desc
Feb 25, 2026
Merged

fix: update app description to reflect accurate location#187
vbuch merged 25 commits intovbuch:mainfrom
bancho22:fix/desc

Conversation

@bancho22
Copy link
Collaborator

No description provided.

@vercel
Copy link

vercel bot commented Feb 18, 2026

Someone is attempting to deploy a commit to the Valery Buchinsky's projects Team on Vercel.

A member of the Team first needs to authorize it.

@vbuch
Copy link
Owner

vbuch commented Feb 18, 2026

It should actually be based on the LOCALITY of the instance. There needs to be a registry to map to that correctly

@bancho22
Copy link
Collaborator Author

Summary of Committed Changes:
10 files changed, 418 insertions(+), 104 deletions(-)

Core Locality Features:
bounds.ts - Added LOCALITY_METADATA registry with getLocalityMetadata(), getLocalityName(), and getLocalityDescription() functions
bounds.test.ts - Comprehensive tests for all locality metadata functions
route.ts - New dynamic manifest route that generates PWA manifest based on NEXT_PUBLIC_LOCALITY
layout.tsx - Updated to use locality-specific description from registry
web/public/manifest.json - Removed (replaced by dynamic route)
Build Infrastructure:
eslint.config.mjs - Added ESLint configuration for the shared package
package.json - Added ESLint dependencies and lint script
tsconfig.json - Updated to include test files in compilation
multi-locality-support.md - Updated documentation with new registry information
pnpm-lock.yaml - Updated with new dependencies
The pre-commit hooks now pass successfully, ensuring code quality across all packages! 🎉

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the app description to be dynamically generated based on the configured locality, moving from a hardcoded description specific to "район Оборище" (Oborishte district) to a more flexible system that supports multiple localities. The change enables the application to serve different localities with appropriate metadata and descriptions.

Changes:

  • Replaced static manifest.json with a dynamic Next.js route handler that generates the manifest based on the NEXT_PUBLIC_LOCALITY environment variable
  • Added locality metadata registry with display names, English names, country codes, and descriptions
  • Updated page metadata in layout.tsx to use locality-specific descriptions from the shared package

Reviewed changes

Copilot reviewed 9 out of 10 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
web/public/manifest.json Deleted static manifest file
web/app/manifest.webmanifest/route.ts Added dynamic manifest generation route with locality-specific metadata
web/app/layout.tsx Updated to use getLocalityDescription() helper and changed manifest reference
shared/src/bounds.ts Added LocalityMetadata interface and helper functions for accessing locality metadata
shared/src/bounds.test.ts Added comprehensive test coverage for new metadata functions
shared/tsconfig.json Removed test file exclusion from build configuration
shared/package.json Added ESLint dependencies and lint script
shared/eslint.config.mjs Added ESLint configuration for the shared package
docs/features/multi-locality-support.md Updated documentation with locality metadata registry information
pnpm-lock.yaml Updated lockfile with new dependencies
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@vercel
Copy link

vercel bot commented Feb 23, 2026

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

Project Deployment Actions Updated (UTC)
oborishte-map Ready Ready Preview, Comment Feb 23, 2026 5:09pm

Copy link
Owner

@vbuch vbuch left a comment

Choose a reason for hiding this comment

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

I'm absolutely happy with it. I've only added a bit of a nitpicky comment. Let's see what Copilot thinks.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 9 out of 10 changed files in this pull request and generated 1 comment.

Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 9 out of 10 changed files in this pull request and generated 4 comments.

Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
vbuch and others added 3 commits February 24, 2026 22:15
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 9 out of 10 changed files in this pull request and generated 7 comments.

Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 11 out of 12 changed files in this pull request and generated 1 comment.

Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 11 out of 12 changed files in this pull request and generated no new comments.

Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@vbuch vbuch merged commit d6beb0c into vbuch:main Feb 25, 2026
3 of 4 checks passed
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.

3 participants