Skip to content

[MOB-12270] new-embedded-view-component#748

Open
lposen wants to merge 7 commits intoemb-ootb/masterfrom
loren/embedded/MOB-12270-new-embedded-view-component
Open

[MOB-12270] new-embedded-view-component#748
lposen wants to merge 7 commits intoemb-ootb/masterfrom
loren/embedded/MOB-12270-new-embedded-view-component

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
Copy link
Member

@joaodordio joaodordio left a comment

Choose a reason for hiding this comment

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

LGTM!

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.

2 participants