Skip to content

Chore/refactor integration display page - #156

Open
armiiller wants to merge 6 commits into
mainfrom
chore/refactor-integration-display
Open

armiiller wants to merge 6 commits into
mainfrom
chore/refactor-integration-display

Conversation

@armiiller

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The remaining Sydney label typo is a non-blocking nit.

Pull request overview

Refactors integration detail pages to use grouped detail components for Webhook and Twilio integrations.

Changes:

  • Adds grouped-detail support and translations.
  • Migrates Webhook and Twilio display partials.
  • Organizes displayed integration options into clearer groups.
File summaries
File Description
config/locales/en.yml Adds grouped detail labels.
app/views/pager_tree/integrations/webhook/v3/_show_options.html.erb Uses grouped Webhook details.
app/views/pager_tree/integrations/live_call_routing/twilio/v3/_show_options.html.erb Groups Twilio options; minor nit: Sydeny should be Sydney.
app/models/pager_tree/integrations/webhook/v3.rb Enables grouped details for Webhook.
app/models/pager_tree/integrations/live_call_routing/twilio/v3.rb Enables grouped details for Twilio.
app/models/pager_tree/integrations/integration.rb Defines the default grouped-detail capability.
Review details
  • Files reviewed: 6/6 changed files
  • Comments generated: 1
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

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.

2 participants