Named entities aren't being associated with their sentences #49
Labels
bug
Something isn't working
Comments
reefdog
added a commit
that referenced
this issue
Mar 28, 2019
Prior to this change, when named entities were created, their appearing sentence wasn’t being associated because we were trying to store its ID in `sentenceId` instead of `sentence_id`, the latter of which is the actual Postgres column name. However, by explictly snake-casing at this point, we may be breaking some of the correct stitching of snake-cased Postgres fields to camelCased object attributes, and so we need to debug whether this is all associated together correctly by Sequelize and queryable by GraphQL. Affects #49
reefdog
added a commit
that referenced
this issue
Mar 28, 2019
Prior to this change, when named entities were created, their appearing sentence wasn’t being associated because we were trying to store its ID in `sentenceId` instead of `sentence_id`, the latter of which is the actual Postgres column name. However, by explicitly snake-casing at this point, we may be breaking some of the correct stitching of snake-cased Postgres fields to camelCased object attributes, and so we need to debug whether this is all associated together correctly by Sequelize and queryable by GraphQL. Affects #49
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
If you add
sentenceIdto the GraphQL query for named entities — or go a step further and try to actually include and display the associated sentence content — you aren't able to. If you then check Postgres to see what's up, you'll notice all named entity'ssentence_idvalues areNULL.Evidently when we create named entities, we aren't associating them with sentences properly.
The text was updated successfully, but these errors were encountered: