Improve Name And Implementation Of `enhanceElementsImages` · guardian/dotcom-rendering

(0 則留言) (0 個反應) (0 位負責人)TypeScript (35 個分叉)auto 404

Pick me upTeam Candidate: WebXgood first issue

倉庫指標

星標: (271 顆星)
PR 合併指標: (PR 指標待抓取)

描述

https://github.com/guardian/dotcom-rendering/blob/5916d46b86dfefbf2a817c93d207516fd114533d/dotcom-rendering/src/model/enhance-images.ts#L448-L457

The name is unclear, perhaps it should be enhanceMainMediaImages.
Why does isPhotoEssay include a check for labs? And why is it different to: https://github.com/guardian/dotcom-rendering/blob/5916d46b86dfefbf2a817c93d207516fd114533d/dotcom-rendering/src/model/enhance-images.ts#L432-L435

貢獻者指南

研究方向: 檢查enhance images.ts中引用的程式碼行，理解當前命名和邏輯。探究為什麼`isPhotoEssay`包含對labs的檢查以及它與另一個函數的不同之處。建議重新命名並統一邏輯。
技術棧: typescript
領域: frontend
議題類型: 重構
難度: 2
預計時間: 1-3 小時
活動狀態: 新近可參與
清晰度: 大致清晰
前置要求: TypeScriptGit
新手友善度: 80