Daniel (dB.) Doubrovkine

icon indicating a website link https://code.dblock.org icon indicating an email [email protected]

icon company @opensearch-project icon location New York, NY icon location @opensearch-project, ex-@aws, former CTO @artsy @ArtsyOpenSource, @vestris, active maintainer of @ruby-grape, hashie, @slack-ruby, @playplayio and more.

Results 3341 comments of Daniel (dB.) Doubrovkine

Array of hashes not failing correctly when array is empty

So the custom validator shouldn't be invoked here, this makes sense to me.

Error in AsValidator when using nested parameters

Please PR a spec + this?

Provide a generic interface for requires: complex type.

I believe https://github.com/intridea/grape/pull/1039 has implemented this, @rnubel would you confirm and I'll close this issue? Is this any different from #993?

Provide a generic interface for requires: complex type.

@birbird this is on HEAD FYI, not in any released version of Grape yet.

Provide a generic interface for requires: complex type.

Maybe we need to do something built-in for Grape entities though ...

custom validator keeping ref to instance variable

@loicginoux Want to rebase and retest? Close or update if the test case is useful and I'll merge.

Stacked versioning broken when not using `:path` version method

@nickrivadeneira could you please PR these tests? Thanks

Stacked versioning broken when not using `:path` version method

I think different versioning schemes may want to behave differently, path versioning may need to be inserted at a different time. I do think you should attempt what you're proposing!...

Password input field

Are you saying that's a swagger 2 feature? There's a [swagger 2](https://github.com/ruby-grape/grape-swagger/tree/swagger-2.0) branch ATM, you could definitely contribute password support change into it.

Password input field

I've labelled this as a new feature.