Slight change to fix narrative images.

[plural]

The :narrative.present? call wasn't quite doing what you would expect in that method and cards without narrative text were still showing narrative image links. Sorry i missed this in the review.

I updated this to use keyword arguments and moved the logic to the caller.

The helper method could be moved to an actual helper in app/helpers/ in a follow-up PR.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.43%. Comparing base (c90825d) to head (8d4fa6c).
Report is 2 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #401      +/-   ##
==========================================
- Coverage   98.44%   98.43%   -0.01%     
==========================================
  Files          76       76              
  Lines        1347     1345       -2     
  Branches      181      181              
==========================================
- Hits         1326     1324       -2     
  Misses         21       21              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[migueldlr]

Looks good - for my general Rails understanding, what's the difference between @object.narrative_text and :narrative_text? Not sure how the scoping works here

[plural]

Looks good - for my general Rails understanding, what's the difference between @object.narrative_text and :narrative_text? Not sure how the scoping works here

@object.attr_name is a Graphiti thing, not a rails thing. Even in a rails model, though, :narrative_text is just a symbol, but narrative_text is the attribute narrative_text.

I debugged this by inspecting the value inside the images function.