shift-org/shift-docs

Change API response when requesting an invalid event ID

Ouverte

#566 ouverte le 24 janv. 2024

Voir sur GitHub
 (1 commentaire) (0 réaction) (1 personne assignée)JavaScript (24 forks)auto 404
APIbackendgood first issue

Métriques du dépôt

Stars
 (28 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

Currently, if you request a single event by ID (e.g. /events.php?id=99999) and the ID is non-existent or the event was deleted, we return a 200 OK response with an empty array. This is essentially a "not found" response, but in a roundabout way. We should just return a proper 404 response.

(But we should continue to return an empty array for day range requests that don't have any events scheduled. An empty array is meaningful in that scenario.)

Guide contributeur

Direction de recherche
Localisez le gestionnaire pour /events.php?id=... et modifiez la réponse pour renvoyer un statut 404 lorsque l'ID de l'événement n'est pas trouvé, tout en conservant un tableau vide pour les requêtes de plage de dates.
Stack technique
phpjavascript
Domaine
backendapi
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitPHP
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75