JSONAPI-Resources / JSONAPI-Resources/jsonapi-resources
Support foreign_type in has_one association
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Ruby
- Estrellas
- 2.3k
- Forks
- 546
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
I am updating an app to use jsonapi:resources and have had to change a model association name without changing the underlying table. This model had a polymorphic field :resource which is now called :item, however the underlying table still has fields resource_id and resource_type.
In my model I can do this:
belongs_to :item, polymorphic: true, foreign_key: "resource_id", foreign_type: "resource_type"
However when I do this in my resource file:
has_one :item, polymorphic: true, foreign_key: 'resource_id', foreign_type: 'resource_type'
I receive an error "undefined method item_type=". From the docs it looks like foreign_key is supported but foreign_type is not. Could this be easily fixed?
Thanks for a great gem, it's been a pleasure to work with!
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.
Línea de trabajo
Comienza con el manejo de la asociación has_one y la compatibilidad existente con foreign_key; después, reproduce las configuraciones de modelo y recurso mostradas en el issue. Se considera terminado cuando foreign_type se acepta sin llamar al método no definido item_type=, con cobertura para asociaciones polimórficas cuyo nombre de modelo difiere de la columna de tipo subyacente.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- rails, ruby
- Área
- api, backend
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 42/100