Daniel (dB.) Doubrovkine comments

Results 3352 comments of


                                            Daniel (dB.) Doubrovkine

Delegate to Roar's represent w/o options argument

Along with README and CHANGELOG make sure to write an UPGRADING with a section about what's affected.

Delegate to Roar's represent w/o options argument

_"but they're hypermedia-ing wrong in that case."_ - I personally feel the opposite way, I wonder what a library like [Hyperclient](https://github.com/codegram/hyperclient) "thinks" when it sees a plain array of hypermedia...

Delegate to Roar's represent w/o options argument

I hear you @jcwilk, I am not saying this is a bad idea, I just see unintended (or intended) consequences that aren't going to make people happy.

Delegate to Roar's represent w/o options argument

Also maybe write a failing spec here for the array representers scenario, I'll try to help fix.

fix: The aggs result of NestedAggregator with sub NestedAggregator may be not accurately

Can @msfroh or @andrross take a look please? Or volunteer another maintainer?

DEPRECATION WARNING: Using `return`, `break` or `throw` to exit a transaction block

Someone who understands what Rails is trying to accomplish here should pitch in ;)

Uninitialized constant error for nested class of Grape::API class when using bootsnap and sidekiqswarm

Thanks for the detailed call outs @allisonphillips! @guigs care to check it out?

Rails 7 has removed ActiveSupport::Dependencies. How should reloading be done now?

> Any updates on this? No updates. AFAIK nobody is working on this.

Rails 7 has removed ActiveSupport::Dependencies. How should reloading be done now?

> It sounds like it's time to move away from grape to something like pure API with VueJS, or Turbo from Hotwire We'll miss you, @blshkv!

Allow lowercase HTTP/2 headers

See https://github.com/ruby-grape/grape/pull/2355 for a beginning of solving this.