You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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.
The text was updated successfully, but these errors were encountered:
SingleShot
pushed a commit
to SingleShot/jsonapi-resources
that referenced
this issue
Oct 10, 2023
This issue is a (choose one):
Checklist before submitting:
Description
We have a project that uses the
jsonapi.rb
gem. We'd like to slowly switch over to usejsonapi-resources
. The former contains a module namedJSONAPI::Rails
. The latter contains a module namedJSONAPI
within which there are multiple uses ofRails
, e.g.class Railtie < Rails::Railtie
andRails.logger
. This leads to a namespace clash andjsonapi-resources
fails to work in such a project, being unable to find anyRails
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 tojsonapi-resources
. In our case, we have a large project and would want to migrate tojsonapi-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 tojsoonapi-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.
The text was updated successfully, but these errors were encountered: