Skip to content

Refactor Webex integration and enhance UI components#21

Merged
adamweeks merged 2 commits intomainfrom
webex-disable
Dec 18, 2025
Merged

Refactor Webex integration and enhance UI components#21
adamweeks merged 2 commits intomainfrom
webex-disable

Conversation

@adamweeks
Copy link
Contributor

  • Updated App component to conditionally load Webex based on route.
  • Modified CreateLobby to include a checkbox for enabling/disabling Webex integration.
  • Enhanced LandingPage with two buttons for launching a game: one for Webex and one for standalone browser.
  • Updated Navbar to display Webex info only on game routes.
  • Adjusted LobbyProvider to preserve the disableWebex query parameter in the lobby URL.
  • Updated useWebex hook to manage Webex loading based on the current route and query parameters.
  • Added tests for new button functionality in LandingPage.

- Updated App component to conditionally load Webex based on route.
- Modified CreateLobby to include a checkbox for enabling/disabling Webex integration.
- Enhanced LandingPage with two buttons for launching a game: one for Webex and one for standalone browser.
- Updated Navbar to display Webex info only on game routes.
- Adjusted LobbyProvider to preserve the disableWebex query parameter in the lobby URL.
- Updated useWebex hook to manage Webex loading based on the current route and query parameters.
- Added tests for new button functionality in LandingPage.
@aws-amplify-us-east-1
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-21.d16sx389cvyqhv.amplifyapp.com

- Cleaned up spacing in App, CreateLobby, and Navbar components for better code clarity.
- Enhanced LandingPage button structure for improved formatting.
- Updated tests for LandingPage to reflect changes in button structure and ensure functionality.
- Adjusted useWebex hook to maintain consistent formatting and improve readability.
@adamweeks adamweeks merged commit 2d7ccdb into main Dec 18, 2025
4 checks passed
@adamweeks adamweeks deleted the webex-disable branch December 18, 2025 19:54
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.

1 participant