Updates, Acars, login callbacks, settings#39
Merged
dev-banane merged 2 commits intomainfrom Oct 19, 2025
Merged
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR introduces authentication callback support, websocket enhancements for real-time controller updates, ACARS settings improvements, and chart drawer functionality upgrades. The changes enhance user experience with flexible post-login navigation, improved controller visibility, and better chart management.
- Added Discord OAuth callback parameter support for post-login redirection
- Enhanced websockets to emit real-time controller updates with VATSIM rating and event controller badges
- Added new ACARS settings for auto-redirect and chart drawer view modes
Reviewed Changes
Copilot reviewed 19 out of 19 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| src/utils/fetch/auth.ts | Added optional callback parameter to Discord OAuth login URL generation |
| src/utils/acars.ts | Updated chart credit attributions to include PFATC co-attribution |
| src/types/settings.ts | Added new ACARS settings for chart view mode and auto-redirect |
| src/sockets/sessionUsersSocket.ts | Changed default position from 'ALL' to 'APP' |
| src/sockets/overviewSocket.ts | Enhanced with controller rating and event controller status |
| src/pages/Submit.tsx | Added auto-redirect to ACARS functionality with settings integration |
| src/pages/Login.tsx | Implemented callback parameter support in login flow |
| src/pages/Home.tsx | Updated developer titles and styling |
| src/pages/Flights.tsx | Added ATIS reminder modal functionality |
| src/pages/ACARS.tsx | Enhanced UI with new buttons, notes expiration, and chart drawer improvements |
| src/components/modals/AtisReminderModal.tsx | Fixed domain URL and updated copy functionality |
| src/components/acars/AcarsTerminal.tsx | Updated terminal title and PDC button styling |
| src/components/acars/AcarsSidebar.tsx | Added controller badges and profile links |
| src/components/acars/AcarsChartDrawer.tsx | Major overhaul with legacy/list view modes and improved UI |
| src/components/Settings/AcarsSettings.tsx | Added chart view mode and auto-redirect settings |
| src/App.tsx | Moved settings route to protected section |
| server/websockets/sessionUsersWebsocket.ts | Added overview data broadcasting on position changes |
| server/websockets/overviewWebsocket.ts | Enhanced with user rating and role data fetching |
| server/routes/auth.ts | Implemented state-based callback parameter handling |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
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.
Authentication Improvements:
callbackparameter in the Discord OAuth flow. The backend now encodes this as state, decodes it on callback, and redirects the user to the specified path if provided. This enables more flexible post-login navigation. [1] [2]Websocket and Real-time Updates:
Frontend Routing:
/settingsroute to be nested under the protected routes section, ensuring only authenticated users can access settings. [1] [2]ACARS Settings and Chart Drawer Enhancements:
AcarsSettingsfor:UI/UX Refinements:
Most Important Changes
Authentication
callbackparameter in Discord OAuth, encoding it in thestateparameter and redirecting users to the specified path after authentication. [1] [2]Websocket/Real-time Updates
Frontend Routing
/settingsroute under protected routes to restrict access to authenticated users. [1] [2]ACARS Settings and Chart Drawer
UI/UX Improvements