David Revelo

Results 23 comments of David Revelo
trafficstars

Hi @stevecrozz , the questions you posted seems quite relevant, let me try to answer them. > User doesn't want to change the notice behavior for all notices (can we...

Hi @stevecrozz , that looks like a nice shortcut to config `ERRBIT_NOTIFY_AT_NOTICES` values. Personally, I would prefer to be able to mute an specific errbit, that said I see your...

Hi @artplan1 this is an interesting addition! I think we need to think about backward compatibility since this header is not going to be present in all DTA versions. Also,...

Hi @NathanielAwoke , have you checked if [this method](https://github.com/graphql-devise/graphql_devise/blob/master/lib/graphql_devise/concerns/controller_methods.rb#L58) is reached? You need to make sure that the controller you are using includes [this concern](https://github.com/graphql-devise/graphql_devise/blob/master/lib/graphql_devise/concerns/set_user_by_token.rb) so gql devise can use...

Hi @TomasBarry , I see your point here, I guess the `graphql_name` could be honored in case it is explicitly set in the class instead of concatenating the resource and...

Hi @maszuDEV , the names of the default operations will use the class and namespace, check these links for reference: - https://github.com/graphql-devise/graphql_devise/blob/e9c9a8f6cf8ad9851261e907347e7a3c8d635e3e/lib/graphql_devise.rb#L40 - https://github.com/graphql-devise/graphql_devise/blob/e9c9a8f6cf8ad9851261e907347e7a3c8d635e3e/lib/graphql_devise/mount_method/operation_preparers/default_operation_preparer.rb#L18 I think a work around for...

Hey @lukefan , I see what you are saying! If you are interested in creating a PR for this I would say a new boolean argument called `mongoid` that will...

That solution is in fact the one we would prefer in the long term, that said it will imply some refactor to make sure code is reused in both generators...