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

(0 件のコメント) (0 件のリアクション) (0 人の担当者)TypeScript (35 件のフォーク)auto 404

Pick me upTeam Candidate: WebXgood first issue

Repository metrics

Stars: (271 個のスター)
PR merge metrics: (PR metrics pending)

説明

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
Issue 種別: リファクタリング
難度: 2
推定時間: 1-3時間
活動状況: 新着
明確さ: おおむね明確
前提条件: TypeScriptGit
初心者向け度: 80