Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: [IOAPPX-396] remove NativeView part #8

Merged
merged 2 commits into from
Oct 14, 2024

Conversation

shadowsheep1
Copy link
Member

@shadowsheep1 shadowsheep1 commented Oct 14, 2024

Short description

This PR removes the NativeView component of the library, which is no longer in use. We have transitioned from utilizing the NativeView to exclusively relying on the NativeModule for native integration. This change simplifies the codebase by eliminating unused components and focuses solely on the module responsible for native functionality.

Key Changes:

  • Removed all instances of the NativeView and related dependencies.
  • Refactored the project to ensure all native functionalities are handled via NativeModule exclusively.
  • Updated documentation (if applicable) to reflect this shift.

Why this change?

The NativeView was no longer being utilized, making it redundant. By removing it, we reduce complexity, improve maintainability, and ensure the library is streamlined around the core functionality provided by NativeModule.

How to test

Run the example app on both Android and iOS physical devices and test a CieID login with IO-BE.

Copy link

github-actions bot commented Oct 14, 2024

Jira Pull Request Link

This Pull Request refers to the following Jira issue IOAPPX-396

@shadowsheep1 shadowsheep1 marked this pull request as ready for review October 14, 2024 13:19
@shadowsheep1 shadowsheep1 requested review from a team as code owners October 14, 2024 13:19
@shadowsheep1 shadowsheep1 changed the title chore: [IOAPPX-396] remove native view part chore: [IOAPPX-396] remove NativeView part Oct 14, 2024
@shadowsheep1 shadowsheep1 merged commit 9afdba1 into main Oct 14, 2024
6 checks passed
@shadowsheep1 shadowsheep1 deleted the remove-native-view-part branch October 14, 2024 15:36
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.

2 participants