Change optional boolean properties, such as `deleted`, to real booleans
Los mantenedores suelen responder en 2 días
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 52/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bastante claro
- Estado de actividad
- Tranquilo
- Stack tecnológico
- python
- Área
- backend-api-design
Línea de trabajo
Localiza las subclases de StripeObject que definen propiedades opcionales como deleted y, a continuación, inspecciona las definiciones de tipos y las pruebas relacionadas. Confirma cómo se representan estas propiedades cuando están ausentes y actualiza las anotaciones y el comportamiento afectados para que los llamadores puedan acceder a ellas como valores booleanos; se considera terminado cuando las pruebas relevantes cubren el acceso directo como booleano sin getattr.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Is your feature request related to a problem? Please describe.
Whenever a StripeObject subclass has the deleted property, it's optional and can ever only be True:
deleted: Optional[Literal[True]]
"""
Always true for a deleted object
"""
This means that if I want to check if a customer was deleted for example, I need to use getattr(customer, "deleted", False) instead of simply customer.deleted (because that could raise an AttributeError).
Describe the solution you'd like
Now that StripeObject isn't simply a subclass of dict anymore, can't the SDK simplify these sort of booleans? Turn them from optional booleans that can only be True to real booleans?
Describe alternatives you've considered
No response
Additional context
No response
- Lenguaje dominante
- Python
- Estrellas
- 2.1k
- Forks
- 538
- Merge medio
- 2 d 18 h
- PR fusionados (30 d)
- 27
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de stripe/stripe-python
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
stripe/stripe-python#1936 · 2 comentarios ·
Los mantenedores suelen responder en 2 días
-
feature-request future
Dificultad 4/5 3-5 días Aptitud para principiantes 55/100
stripe/stripe-python#1941 · 4 comentarios ·
Los mantenedores suelen responder en 2 días
-
Add support for HTTPX2Abiertofeature-request future
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
stripe/stripe-python#1886 · 1 comentario ·
Los mantenedores suelen responder en 2 días
-
feature-request
Dificultad 2/5 1-3 horas Aptitud para principiantes 54/100
stripe/stripe-python#1841 · 1 comentario ·
Los mantenedores suelen responder en 2 días
-
Add async context manager support for AIOHTTPClient and HTTPXClientQuizá libre de nuevo Un pull request para esta issue se cerró sin fusionarse. Abiertofeature-request future
Dificultad 3/5 1-2 días Aptitud para principiantes 74/100
stripe/stripe-python#1796 · 3 comentarios ·
Los mantenedores suelen responder en 2 días
Todos los issues de stripe/stripe-python
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
UKGovernmentBEIS/inspect_ai#5802 ·
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
no-human-ai/no_human#660 ·
Los mantenedores suelen responder en 1 día
-
Add a Security Insights v2 fileAbiertodocumentation good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
documentation need help question
Dificultad 1/5 1-3 horas Aptitud para principiantes 66/100
phonology024/babelscribe#26 ·
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100