Conversation
✅ Deploy Preview for gr-mapache ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
I think the
|
There was a problem hiding this comment.
Pull request overview
This pull request adds a temporary Settings page to prevent navigation errors when users click the Settings link in the sidebar. The implementation follows the same pattern as the existing ChatPage, providing a placeholder page until the full settings functionality is implemented.
Key changes:
- Created a new SettingsPage component with placeholder content
- Added the /settings route to the application router
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| dashboard/src/pages/settings/SettingsPage.tsx | New placeholder Settings page component following the same pattern as ChatPage |
| dashboard/src/main.tsx | Added SettingsPage import and /settings route configuration |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
wow copilot fucking sucks |
yea this is a security thing w Sentinel. Since netlify deploy previews use a random preview url we the oauth exchange fails (invalid redirect_uri). couple things we can do; use a custom subdomain for the previews and in theory we can wildcard that domain as the redirect uri (not totally sure if netlify allows this/cost), or we can change how sentinel works to allow insecure redirect_uri for specific applications (mapache-dev, for ex). cc @austinchan3678 just to keep u in the know |
2 participants
Adds a missing temporary settings tab similar to the chat tab, this means that if you click the
Settingsin the sidebar it will no longer throw you off the site.