fix the condition for rendering old backlog list

[toy]

Ticket

https://community.openproject.org/wp/73448

What are you trying to accomplish?

See eac7243 #22721

Merge checklist

• Added/updated tests
• Added/updated documentation in Lookbook (patterns, previews, etc)
• Tested major browsers (Chrome, Firefox, Edge, ...)

[Copilot]

Pull request overview

Adjusts Backlogs controller rendering so Turbo Frame requests only return the legacy backlog list partial when the request targets the backlogs_container frame, preventing mismatched frame responses (and subsequent full-page reload fallbacks) in other Turbo interactions.

Changes:

• Replace broad turbo_frame_request? check with a turbo_frame_request_id match for backlogs_container in the legacy index action.
• Ensure non-backlogs_container Turbo Frame requests receive the full :index render path.

Comments suppressed due to low confidence (1)

modules/backlogs/app/controllers/rb_master_backlogs_controller.rb:60

• The updated branching introduces new behavior for Turbo Frame requests whose frame id is not "backlogs_container" (they now render the full :index view instead of the list partial). There is request spec coverage for the "backlogs_container" frame, but no coverage for a different Turbo-Frame header (e.g., "content-bodyRight"), which is the regression scenario this change is meant to address. Please add a request spec asserting that a non-backlogs_container turbo-frame request renders :index (and does not render the _list partial).

    case turbo_frame_request_id
    when "backlogs_container"
      render partial: "list", layout: false
    else
      render :index
    end