Skip to content

Conversation

@lposen
Copy link
Contributor

@lposen lposen commented Oct 13, 2025

🔹 JIRA Ticket(s) if any

✏️ Description

Added embedded view component

Testing

  • Start the app (see instructions under "start the app")
  • Sign in
  • Click on "Embedded"
  • Click "Sync Messages"
  • Select a View Type
  • Click "Get Messages"
  • The correct view type should be shown in the messages below
  • Change the view type
  • The new view type should show in the messages
Banner Card Notification
Screenshot 2026-02-10 at 3 22 54 PM Screenshot 2026-02-10 at 3 23 00 PM Screenshot 2026-02-10 at 3 23 03 PM

Start the app

  • In the root, run yarn install
  • cd into example
    • run yarn install
    • Run the rn server in one tab:
      • run yarn start --reset-cache
      • Do not close this tab
    • Create a new tab in example and run yarn android
    • Create a new tab in example and run yarn ios (or open the workspace and run with xcode)

@github-actions
Copy link

github-actions bot commented Oct 13, 2025

Lines Statements Branches Functions
Coverage: 62%
62.53% (409/654) 40.97% (109/266) 60.25% (141/234)

@qltysh
Copy link

qltysh bot commented Oct 13, 2025

Qlty

Coverage Impact

⬇️ Merging this pull request will decrease total coverage on emb-ootb/master by 0.54%.

Modified Files with Diff Coverage (4)

RatingFile% DiffUncovered Line #s
New file Coverage rating: F
src/embedded/components/IterableEmbeddedBanner.tsx20.0%10-14
New file Coverage rating: A
src/embedded/components/IterableEmbeddedView.tsx100.0%
New file Coverage rating: F
src/embedded/components/IterableEmbeddedNotification.tsx20.0%10-17
New file Coverage rating: F
src/embedded/components/IterableEmbeddedCard.tsx20.0%9-13
Total47.8%
🤖 Increase coverage with AI coding...

In the `loren/embedded/MOB-12270-new-embedded-view-component` branch, add test coverage for this new code:

- `src/embedded/components/IterableEmbeddedBanner.tsx` -- Line 10-14
- `src/embedded/components/IterableEmbeddedCard.tsx` -- Line 9-13
- `src/embedded/components/IterableEmbeddedNotification.tsx` -- Line 10-17

🚦 See full report on Qlty Cloud »

🛟 Help
  • Diff Coverage: Coverage for added or modified lines of code (excludes deleted files). Learn more.

  • Total Coverage: Coverage for the whole repository, calculated as the sum of all File Coverage. Learn more.

  • File Coverage: Covered Lines divided by Covered Lines plus Missed Lines. (Excludes non-executable lines including blank lines and comments.)

    • Indirect Changes: Changes to File Coverage for files that were not modified in this PR. Learn more.

@qltysh
Copy link

qltysh bot commented Oct 13, 2025

5 new issues

Tool Category Rule Count
qlty Duplication Found 20 lines of similar code in 3 locations (mass = 75) 3
qlty Structure Function with many returns (count = 5): IterableEmbeddedView 1
qlty Structure Function with high complexity (count = 5): IterableEmbeddedView 1

@lposen lposen added the embedded Issues/PRs related to Embedded Messages label Oct 14, 2025
Base automatically changed from loren/embedded/MOB-12268-android-add-click-handling-and-track to loren/embedded/master December 24, 2025 19:39
}
}, [viewType]);

return Cmp ? <Cmp {...props} /> : null;
Copy link

Choose a reason for hiding this comment

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

Found 2 issues:

1. Function with many returns (count = 5): IterableEmbeddedView [qlty:return-statements]


2. Function with high complexity (count = 5): IterableEmbeddedView [qlty:function-complexity]

Base automatically changed from loren/embedded/master to master January 29, 2026 21:32
<Text>IterableEmbeddedBanner</Text>
</View>
);
};
Copy link

Choose a reason for hiding this comment

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

Found 20 lines of similar code in 3 locations (mass = 75) [qlty:similar-code]

<Text>IterableEmbeddedCard</Text>
</View>
);
};
Copy link

Choose a reason for hiding this comment

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

Found 19 lines of similar code in 3 locations (mass = 75) [qlty:similar-code]

<Text>IterableEmbeddedNotification</Text>
</View>
);
};
Copy link

Choose a reason for hiding this comment

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

Found 23 lines of similar code in 3 locations (mass = 75) [qlty:similar-code]

@lposen lposen changed the base branch from master to emb-ootb/master February 10, 2026 23:13
@lposen lposen marked this pull request as ready for review February 10, 2026 23:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

embedded Issues/PRs related to Embedded Messages

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant