Skip to content

Conversation

@hakansa
Copy link
Contributor

@hakansa hakansa commented Aug 20, 2025

No description provided.

Copilot AI review requested due to automatic review settings August 20, 2025 09:46
Copy link

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 adds a skip_auto_apply feature to exit nodes, allowing users to control whether exit nodes are automatically applied to distribution groups. The feature includes UI controls, table display, and visual indicators for the auto-apply status.

  • Adds skip_auto_apply boolean field to Route interface and related API calls
  • Introduces "Auto Apply Route" toggle switch in route creation and update modals for exit nodes
  • Adds new table column and cell component to display auto-apply status in route tables

Reviewed Changes

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

Show a summary per file
File Description
src/interfaces/Route.ts Adds skip_auto_apply field to Route interface
src/modules/routes/RouteUpdateModal.tsx Adds Auto Apply toggle for exit nodes in update modal
src/modules/routes/RouteModal.tsx Adds Auto Apply toggle for exit nodes in create modal
src/modules/routes/RouteAutoApplyCell.tsx New component for displaying/toggling auto-apply status in table
src/modules/routes/RouteTable.tsx Adds Auto Apply column to route table
src/modules/exit-node/useHasExitNodes.tsx Updates hook to return exit node info including skip_auto_apply status
src/modules/exit-node/ExitNodePeerIndicator.tsx Updates indicator with different colors based on auto-apply status
src/modules/exit-node/ExitNodeDropdownButton.tsx Updates to use new exit node info structure
src/modules/peer/PeerNetworkRoutesSection.tsx Updates to use new exit node info structure
src/contexts/RoutesProvider.tsx Adds skip_auto_apply field to route creation and update operations
Comments suppressed due to low confidence (1)

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@CLAassistant
Copy link

CLAassistant commented Aug 20, 2025

CLA assistant check
All committers have signed the CLA.

@heisbrot heisbrot merged commit 5d4e491 into main Aug 20, 2025
3 checks passed
@heisbrot heisbrot deleted the feat/ex-no-autoapply branch August 20, 2025 11:11
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.

4 participants