-
Notifications
You must be signed in to change notification settings - Fork 3
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
mc/in-app-purchases-example #5
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very nice work here, just some minor comments. Also, will love for some more detailed (step by step) explanation of the store kit integration and the purchase verification, meaning, more comments on the code and an step by step explanation of this in the README.
InAppPurchasesAtlasAppServices/FoodieFolio/FoodieFolio/Managers/AuthenticationManager.swift
Outdated
Show resolved
Hide resolved
InAppPurchasesAtlasAppServices/FoodieFolio/FoodieFolio/Views/AccountView.swift
Show resolved
Hide resolved
InAppPurchasesAtlasAppServices/FoodieFolio/FoodieFolio/ContentView.swift
Outdated
Show resolved
Hide resolved
InAppPurchasesAtlasAppServices/FoodieFolio/FoodieFolio.xcodeproj/project.pbxproj
Show resolved
Hide resolved
InAppPurchasesAtlasAppServices/FoodieFolio/FoodieFolio/Managers/RealmManager.swift
Outdated
Show resolved
Hide resolved
InAppPurchasesAtlasAppServices/FoodieFolio/FoodieFolio/Extensions/TextField+Extension.swift
Outdated
Show resolved
Hide resolved
InAppPurchasesAtlasAppServices/FoodieFolio/FoodieFolio/Managers/StoreKitManager.swift
Outdated
Show resolved
Hide resolved
InAppPurchasesAtlasAppServices/FoodieFolio/FoodieFolio/Managers/StoreKitManager.swift
Outdated
Show resolved
Hide resolved
InAppPurchasesAtlasAppServices/FoodieFolio/FoodieFolio/Managers/StoreKitManager.swift
Show resolved
Hide resolved
InAppPurchasesAtlasAppServices/FoodieFolio/FoodieFolio/Managers/StoreKitManager.swift
Show resolved
Hide resolved
…s/RealmManager.swift Co-authored-by: Diana Perez Afanador <[email protected]>
…s/StoreKitManager.swift Correct typo Co-authored-by: Diana Perez Afanador <[email protected]>
…swift-samples into mc/in_app_purchases_example
…t to the List to improve readability
…er terminology. Added category field on Recipe schema
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!. Excellent work with this example, just some minor comments before merging
|
||
This example contains a skeleton app to be used as a reference on how to In App Purchase with a seamless integration in App Services using the [Swift SDK](https://www.mongodb.com/docs/realm/sdk/swift/#realm-swift-sdk). | ||
In this project you will be able to see an example of the following use case: | ||
* Authenticate users using the Swift SDK. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm missing some point explaining with integrate with store kit and what we do with it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've updated it now with more details. Let me know if it's better or if you want more detailed information.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's ok, I thinks that's all the comments solved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice, thank you for all the help and the review! I'll go ahead and squash and merge
No description provided.