elastic/kibana

[Response Ops][Alerting] Restructure actions FT route definition

Open

#179.086 aperta il 20 mar 2024

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)TypeScript (8021 fork)batch import
Feature:ActionsTeam:ResponseOpsgood first issuetechnical debt

Metriche repository

Star
 (19.065 star)
Metriche merge PR
 (Merge medio 1g 16h) (999 PR mergiate in 30 g)

Descrizione

Suggestion from PR review for https://github.com/elastic/kibana/pull/178682 to restructure the way routes are defined in the actions FT plugin. Referencing the route definitions in x-pack/test/alerting_api_integration/common/plugins/actions_simulators/server/unsecured_actions_simulation.ts

should we structure this like we do pagerduty, so we don't hardcode the `/api/...` path - which in theory, someday, could become a real API (but doubtful).  Eg, 

- https://github.com/elastic/kibana/blob/main/x-pack/test/alerting_api_integration/common/plugins/actions_simulators/server/pagerduty_simulation.ts
- https://github.com/elastic/kibana/blob/main/x-pack/test/alerting_api_integration/common/plugins/actions_simulators/server/plugin.ts

This is mostly pre-existing code, so I'd say we could just create a cleanup issue for it as well ...

Originally posted by @pmuellr in https://github.com/elastic/kibana/pull/178682#discussion_r1532077936

Guida contributor

Direzione di ricerca
Esamina il modello di definizione delle route della simulazione pagerduty di riferimento e applicalo al file unsecured actions simulation.ts. Ristruttura le definizioni delle route per evitare di codificare il percorso `/api/`.
Tech stack
typescript
Dominio
backendtesting
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
TypeScriptKibana plugin structure
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70