[2.x] Experiment with removing the mediaLink method

[emmadesilva]

Part of #1904 and removes the mediaLink helper as it can be merged into the main asset() helper.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Please upload report for BASE (normalize-the-asset-api@314c57a). Learn more about missing BASE report.

Additional details and impacted files

@@                     Coverage Diff                     @@
##             normalize-the-asset-api     #1914   +/-   ##
===========================================================
  Coverage                           ?   100.00%           
  Complexity                         ?      1871           
===========================================================
  Files                              ?       192           
  Lines                              ?      4942           
  Branches                           ?         0           
===========================================================
  Hits                               ?      4942           
  Misses                             ?         0           
  Partials                           ?         0           

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

[emmadesilva]

I'm starting to remember why we have two methods. Media link was intended to be the main way to use get assets, as it uses the more compatible relative links. The second parameter was for things where specs required absolute urls like metadata. But I did not document this so I forgot it.

[emmadesilva]

Drafting until I consider what solves the right problems the best way.

[emmadesilva]

Drafting until I consider what solves the right problems the best way.

Maybe we should improve the object API by adding path() and link() instance helpers.

[emmadesilva]

Superseded by #1932