Differentiate Task request schemas where different properties are required
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Idoneità per principianti
- 25/100
- Tipo di issue
- Funzionalità
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Ferma
- Stack tecnologico
- typescript
- Ambito
- api
Direzione di ricerca
Inizia con le modifiche allo schema discusse in pull request 591 e con l’output generato in src/client. Esegui yarn generate, quindi esamina src/tasks/containers/TaskPage.test.tsx intorno alla riga 259 per comprendere l’errore di tipo attuale. È fatto quando il client generato espone le proprietà differenziate della richiesta Task e i test UI esistenti superano il controllo dei tipi.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
For cloud, a Task must have one of flux or scriptID. If it has scriptID, then it may have other fields (that aren't compatible with flux in a create request).
Differentiating the schemas with oneOf breaks the current UI tests--e.g.:
@jstirnaman
src/tasks/containers/TaskPage.test.tsx:259:66 - error TS2339: Property 'flux' does not exist on type 'TaskCreateRequest | TaskWithScriptCreateRequest'. Property 'flux' does not exist on type 'TaskWithScriptCreateRequest'.
I'm assuming it needs a discriminator to know which schema to use? I ran yarn generate to generate src/client/* locally and I don't even see TaskWithScriptCreateRequest in the results, so I 'm confused.
I'll undo this for now and move it to my someday list. It seems beneficial to clients if they could discern which properties are compatible and required. I hate having it in human-readable-only fields.
Originally posted by @jstirnaman in https://github.com/influxdata/openapi/pull/591#discussion_r1010901842
- Lingua principale
- Shell
- Stelle
- 18
- Fork
- 9
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
Non abbiamo ancora controllato i file di configurazione di questo progetto. Parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di influxdata/openapi
-
documentation
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
influxdata/openapi#657 ·
-
Difficoltà 1/5 1-3 ore Idoneità per principianti 85/100
influxdata/openapi#653 ·
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 65/100
influxdata/openapi#240 ·
-
Update Routes.yml for backup, restore, labels, notificationRules, notificationEndpoints, scrapers, swagger, checks, plugins, delete, readyForse di nuovo libera @devanbenz l’ha presa 140 giorni fa e non c’è nessuna pull request aperta. Apertabug
influxdata/openapi#654 · 1 assegnatario ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 25/100
influxdata/openapi#636 ·
Tutte le issue di influxdata/openapi
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
I maintainer di solito rispondono entro 1 giorno
-
Update vtm to 2026.9.26Apertapackage-update
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 78/100
oSoWoSo/vOid_Community_repOsitory#195 · 1 commento ·
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 92/100
obra/superpowers#2385 ·
I maintainer di solito rispondono entro 2 giorni
-
follow-up
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
yschimke/homeassistant-remotecompose#680 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
dehydrated-io/dehydrated#1007 ·