Skip to content

Conversation

@chayelheinsen
Copy link
Contributor

Adds support for Tactics and DaVinci enabled events

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR bumps the version to 2.0.0 and adds support for Tactics and DaVinci enabled events while extending type definitions and updating dependencies.

  • Introduces a new Currency enum and enhances product/order interfaces with currency and updated price/revenue types.
  • Updates dependency versions in the podspec, Android build.gradle, and package.json.
  • Revises example usage in App.tsx to incorporate the new Currency type and updated pricing schema.

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
src/index.tsx Adds Currency enum and updates Product and Order interfaces.
react-native-movable-ink.podspec Updates dependency version and includes local testing comments.
package.json Bumps SDK version and refines keywords.
example/src/App.tsx Adjusts sample events to use the new Currency type and pricing format.
android/build.gradle Updates MovableInk SDK dependency version.
.tool-versions Updates Node.js version.
Comments suppressed due to low confidence (1)

src/index.tsx:4

  • Consider adding a brief comment above the Currency enum to explain its purpose and its relation to the new event support, which helps maintain clarity for future developers.
export enum Currency {

chayelheinsen and others added 3 commits May 7, 2025 12:04
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@chayelheinsen chayelheinsen marked this pull request as ready for review May 9, 2025 14:09
@chayelheinsen chayelheinsen merged commit 27a4d85 into main May 9, 2025
4 checks passed
@chayelheinsen chayelheinsen deleted the chayel/sc-152528/v2-release branch May 9, 2025 17:05
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