enhancemnt to "add_root" feature
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- ruby
- Área
- api, documentation
Línea de trabajo
Start at the add_root feature entry point and inspect how it derives the root from the response model class; then check whether the response model exposes root_element(root_type) for collection_root and root. Done when generated Swagger output uses the entity-configured singular and plural roots while preserving existing behavior.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
The new add_root feature is extremely nice and something I've been waiting for in this gem for a while.
One small issue with it is that the way it handles dynamically building the 'root' name is solely based on the response model class name. Where as, ideally what would be nice is to "detect" if said response model is a grape-entity class and query the "Root" configuration from that class.
I have several GrapeEntity classes that change the "root" to be someone slightly different than the internal class name (e.g. OrganizationMember -> 'members'/'member') so having the swagger gem consult the entity class. (e.g. simply calling OrganizationMember.root_element(root_type) where root_type is :collection_root or :root ).
- Lenguaje dominante
- Ruby
- Estrellas
- 1.1k
- Forks
- 477
- Merge medio
- 1 d 18 h
- PR fusionados (30 d)
- 5
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin 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 ruby-grape/grape-swagger
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
ruby-grape/grape-swagger#1014 ·
Los mantenedores suelen responder en 1 día
-
Support default_response and default in 2.3; stop reading route.descPosiblemente ocupada @numbata la tomó hace 59 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
ruby-grape/grape-swagger#989 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
ruby-grape/grape-swagger#987 · 3 comentarios ·
Los mantenedores suelen responder en 1 día
-
Bug report: omitted array parameters with `array_use_braces: true`Posiblemente ocupada @olivier-thatch la tomó hace 533 días. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
ruby-grape/grape-swagger#952 · 2 comentarios · 1 reacción ·
Los mantenedores suelen responder en 1 día
-
Bug Report: `default` for array params (`type: [String]`) is incorrectly placed inside `items`Posiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
ruby-grape/grape-swagger#950 · 2 comentarios · 1 reacción ·
Los mantenedores suelen responder en 1 día
Todos los issues de ruby-grape/grape-swagger
Issues similares
-
allow favicon to be served from CDN/CORS (i.e. treat it the same as the avatar)Posiblemente ocupada @enkaskal la tomó hoy. Abiertoenhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
cotes2020/jekyll-theme-chirpy#2811 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
-
Add a dark modeAbiertoaccessibility enhancement good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
samiromer2/Ruby-Consent-Portal#35 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
-
bug detail
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
keygen-sh/keygen-api#1161 ·
Los mantenedores suelen responder en 1 día