Skip to content

Refactor NOTICKET Rename Story Cell#33117

Open
adudenamedruby wants to merge 1 commit intomainfrom
rgb/dozing-imaginative-banjo
Open

Refactor NOTICKET Rename Story Cell#33117
adudenamedruby wants to merge 1 commit intomainfrom
rgb/dozing-imaginative-banjo

Conversation

@adudenamedruby
Copy link
Copy Markdown
Contributor

@adudenamedruby adudenamedruby commented Apr 15, 2026

💡 Description

Creating a ticket for this work takes longer than the work itself.

📝 Checklist

  • I filled in the ticket numbers and a description of my work
  • I updated the PR name to follow our PR naming guidelines
  • I ensured unit tests pass and wrote tests for new code
  • If working on UI, I checked and implemented accessibility (Dynamic Text and VoiceOver)
  • If adding telemetry, I read the data stewardship requirements and will request a data review
  • If adding or modifying strings, I read the guidelines and will request a string review from l10n
  • If needed, I updated documentation and added comments to complex code

@adudenamedruby adudenamedruby requested a review from Foxbolts April 15, 2026 20:48
@adudenamedruby adudenamedruby requested a review from a team as a code owner April 15, 2026 20:48
Copy link
Copy Markdown
Collaborator

@Foxbolts Foxbolts left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we update the comment at the top of MerinoStoryConfiguration and MerinoStoryResponse as well

@mobiletest-ci-bot
Copy link
Copy Markdown

Messages
📖 Project coverage: 41.17%

🧹 Tidy commit

Just 4 file(s) touched. Thanks for keeping it clean and review-friendly!

✅ New file code coverage

No new file detected so code coverage gate wasn't ran.

Client.app: Coverage: 39.42

File Coverage
HomepageDiffableDataSource.swift 84.4%
StoryCell.swift 0.0% ⚠️
HomepageViewController.swift 47.92% ⚠️

Generated by 🚫 Danger Swift against 150596f

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants