refactor: remove __ping__ request
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 72/100
Direzione di ricerca
Inizia in R/utils.R, in check_connect_license(), e in R/connect.R, in Connect$raise_error(), per tracciare la gestione attuale di 402. Rimuovi la richiesta ping, centralizza il relativo messaggio speciale in raise_error() ed elimina i vari file mock di test ping.json. Il lavoro è completato quando gli errori relativi a licenze non valide mantengono il messaggio speciale senza una richiesta aggiuntiva e i test interessati hanno esito positivo.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
check_connect_license() calls the __ping__ endpoint and looks for 402 response to indicate that the license is not valid. But we don't need to do that because any request will error that way if the license is invalid. So just move the special messaging for 402 errors into Connect$raise_error() and delete this unnecessary server request.
Also delete the various __ping__.json test mock files, as they are no longer needed.
- Lingua principale
- R
- Stelle
- 54
- Fork
- 27
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
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 posit-dev/connectapi
-
Move uuid to SuggestsAperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 62/100
posit-dev/connectapi#249 ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 65/100
posit-dev/connectapi#527 ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
posit-dev/connectapi#526 · 1 reazione ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 48/100
posit-dev/connectapi#506 ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 48/100
posit-dev/connectapi#486 ·
Tutte le issue di posit-dev/connectapi
Issue simili
-
Write an exception for delayed NYC data onlyForse già presa @kaitejohnson l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
-
New `filter_out()` requires dplyr >= 1.2.0Forse già presa @jameslairdsmith l’ha presa 1 giorno fa. Aperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 74/100
-
Add smvr alongside semverAperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 78/100