Literals in Relay createConnectionResolver({...}) where param stopped working

[intellix]

Sequelize 4.44

Got something like this:

relay.createConnectionResolver({
  target: () => BoxItem,
  ignoreArgs: { ...connectionArgs, orderBy: true },
  where: () => {
    return {
      boxId: {
        [Op.eq]: sequelize.literal(`(SELECT "id" FROM "Box" WHERE "slug" = 'abc' and "marketId" = 1)`),
      },
    };
  },
});

Caused by this commit with replaceWhereOperators: https://github.com/mickhansen/graphql-sequelize/commit/3cb923f0799092df15cccd033363be7ef7730e97

When stepping through the debugger it seems like the type is lost:

Before:

After:

[intellix]

I'm not sure if this should work really. I can shove all of my logic into the before hook instead the where param. I don't know why I put it there in the first place.

[mickhansen]

Since we still support where it probably should work, not exactly sure how it broke, perhaps @Azerothian can lend some assistance.

[mickhansen]

I personally only use before, i just find it easier when you have args that might influence eachother etc.

[intellix]

Thanks :) I've moved all logic from where to before. Not sure why I used it before

[mickhansen]

where was what i thought would be a clever idea, but in practice it sees little effective use.