Skip to content

Conversation

@ericproulx
Copy link
Contributor

@ericproulx ericproulx commented Dec 7, 2025

This PR fixes the call to cascade when read from the api.

The following test was failing

describe '.cascade' do
  subject { api.cascade }
  let(:api) do 
    Class.new(Grape::API) do
      cascade true
    end
  end

  it { is_expected.to eq(true)}
end

All make proper changes after #2633 is merged since the latter is moving the function in routing module

@ericproulx ericproulx requested a review from dblock December 7, 2025 22:22
@ericproulx ericproulx force-pushed the fix/cascade_reading_api_level branch from a54a4bd to ab9e9cc Compare December 7, 2025 22:26
@dblock
Copy link
Member

dblock commented Dec 9, 2025

Merged #2633

@ericproulx ericproulx force-pushed the fix/cascade_reading_api_level branch from ab9e9cc to c36690b Compare December 10, 2025 17:01
@ericproulx ericproulx force-pushed the fix/cascade_reading_api_level branch from c36690b to 900a716 Compare December 11, 2025 19:38
@dblock dblock merged commit 927d55a into master Dec 11, 2025
108 of 109 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants