Skip to content

Conversation

@alisa911
Copy link
Contributor

@alisa911 alisa911 commented Jan 8, 2026

No description provided.

@alisa911
Copy link
Contributor Author

alisa911 commented Jan 8, 2026

@alisa911 alisa911 marked this pull request as ready for review January 8, 2026 14:19
@alisa911 alisa911 requested a review from Copilot January 8, 2026 14:19
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 pull request adds pricing page redirection for unauthorized users attempting to save routes to the cloud. The key change enables free account users and logged-out users to access the "Save to Cloud" button, which now redirects them to the pricing page instead of being hidden.

Key changes:

  • Save to Cloud functionality now redirects unauthorized users to pricing page instead of hiding the button
  • Added location search API fallback for navigation point inputs (allowing text-based location search in addition to coordinates)
  • Close navigation settings when opening route details or login menu for better UX

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
NavigationMenu.jsx Added pricing page redirect for unauthorized save-to-cloud attempts; made button always visible
NavigationPointsManager.jsx Made point input parsing async with API search fallback for location names
RouteSummaryCard.jsx Passed total distance from API to graph component
NavigationSummaryGraph.jsx Added totalDistanceMeters parameter for accurate distance display
LoginManager.js Close navigation settings when opening login menu

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

@alisa911 alisa911 requested a review from RZR-UA January 8, 2026 14:25
@alisa911 alisa911 merged commit e3952e1 into main Jan 9, 2026
@alisa911 alisa911 deleted the 080126_3 branch January 9, 2026 07:33
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