Rails Namespace Conflict with jsonapi.rb
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 35/100
Línea de trabajo
Comienza localizando las referencias a Rails dentro del espacio de nombres JSONAPI, especialmente la herencia de Railtie y el uso del logger descritos en el issue, y reproduce el conflicto junto con el gem jsonapi.rb. El trabajo estará terminado cuando la biblioteca resuelva los métodos de Rails previstos sin un conflicto de espacios de nombres y el comportamiento existente permanezca intacto.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
This issue is a (choose one):
- Problem/bug report.
- Feature request.
- Request for support. Note: Please try to avoid submitting issues for support requests. Use Gitter instead.
Checklist before submitting:
- I've searched for an existing issue.
- I've asked my question on Gitter and have not received a satisfactory answer.
- I've included a complete bug report template. This step helps us and allows us to see the bug without trying to reproduce the problem from your description. It helps you because you will frequently detect if it's a problem specific to your project.
- The feature I'm asking for is compliant with the JSON:API spec.
Description
We have a project that uses the jsonapi.rb gem. We'd like to slowly switch over to use jsonapi-resources. The former contains a module named JSONAPI::Rails. The latter contains a module named JSONAPI within which there are multiple uses of Rails, e.g. class Railtie < Rails::Railtie and Rails.logger. This leads to a namespace clash and jsonapi-resources fails to work in such a project, being unable to find any Rails methods, etc.
While it can be argued that this is not a bug in jsonapi-resources, it would probably be good to avoid this namespace clash if easy, especially when the clash is with another JSON:API gem. Fixing this would make it easier for projects to evaluate and switch over to jsonapi-resources. In our case, we have a large project and would want to migrate to jsonapi-resources piece by piece, but currently must do so wholesale.
A simple and modest fix would be to fully scope references to Rails, e.g. ::Rails::Railtie and ::Rails.logger. This is a safe change to jsoonapi-resources
Sorry - I could not get the bug template to execute successfully. If you are willing to accept this "bug" report I will submit an MR.
- Lenguaje dominante
- Ruby
- Estrellas
- 2.3k
- Forks
- 546
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
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 JSONAPI-Resources/jsonapi-resources
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
-
ArgumentError with Rails 8.1 Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 50/100
JSONAPI-Resources/jsonapi-resources#1488 · 1 comentario ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 42/100
JSONAPI-Resources/jsonapi-resources#1479 · 2 comentarios ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 30/100
JSONAPI-Resources/jsonapi-resources#1477 · 3 comentarios · 1 reacción ·
-
Fields filtering with include Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
Todos los issues de JSONAPI-Resources/jsonapi-resources
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
simp/pupmod-simp-simp#395 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 80/100
simp/pupmod-simp-rsyslog#219 ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
simp/pupmod-simp-pupmod#256 ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
simp/pupmod-simp-sudo#150 ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100