Skip to content

Implement automatic trial conversion detection + add productID param #234

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

Merged
merged 5 commits into from
Mar 11, 2025

Conversation

Jeehut
Copy link
Contributor

@Jeehut Jeehut commented Feb 25, 2025

Closes https://github.com/TelemetryDeck/PirateMetrics/issues/4.

Besides implementing the requirements in above linked task, I also updated the currency conversion rates.

@Jeehut Jeehut force-pushed the feature/free-trials branch from c39dffe to 26d75ac Compare February 25, 2025 14:37
@Jeehut Jeehut force-pushed the feature/free-trials branch from 26d75ac to fb34c65 Compare February 25, 2025 14:38
@Jeehut Jeehut requested a review from winsmith February 25, 2025 14:40
@Jeehut
Copy link
Contributor Author

Jeehut commented Mar 7, 2025

@winsmith Just a reminder – this is still awaiting your review. 😊

Copy link
Contributor

@winsmith winsmith left a comment

Choose a reason for hiding this comment

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

Love it!

@Jeehut Jeehut merged commit 82c10d3 into main Mar 11, 2025
6 checks passed
@Jeehut Jeehut deleted the feature/free-trials branch March 11, 2025 14:26
}
}

return priceValueInUSD
}

private static let currencyCodeToOneUSDExchangeRate: [String: Double] = [
Copy link
Contributor

Choose a reason for hiding this comment

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

@Jeehut @winsmith are we using a specific source for the currency rates? How often do we plan to have these updated?

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