Skip to content

Conversation

@ThomasSession
Copy link
Collaborator

Wiring the server data to our presentation models.
Removed the SubscriptionDetails in favour of the server data directly.
Also removed some code from the subscriber in favour of the payment metadata.

A successful google subscription purchase now links up to our server endpoint.

Cleaned up compose previews.

Copy link
Collaborator

@SessionHero01 SessionHero01 left a comment

Choose a reason for hiding this comment

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

Nice work!

data class Item(
val status: ProDetailsStatus,
@SerialName("plan")
val planDuration: ServerPlanDuration,
Copy link
Collaborator

Choose a reason for hiding this comment

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

So this is a mandatory property? Once added there's no going back...

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I'll check with the team, though I'm not sure what it would mean for it to be missing

val validUntil: Instant
val duration: ProSubscriptionDuration
val subscriptionDetails: SubscriptionDetails
val subscriptionDetails: PaymentProviderMetadata
Copy link
Collaborator

Choose a reason for hiding this comment

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

Should this be renamed?

@ThomasSession ThomasSession merged commit d994bea into dev Nov 19, 2025
4 checks passed
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