-
Notifications
You must be signed in to change notification settings - Fork 15
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
Improve docs part 2 #131
Improve docs part 2 #131
Conversation
public delegate void LogInFunc(CustomerInfo customerInfo, bool created, Error error); | ||
|
||
public delegate void GetOfferingsFunc(Offerings offerings, Error error); | ||
|
||
public delegate void CheckTrialOrIntroductoryPriceEligibilityFunc(Dictionary<string, IntroEligibility> products); |
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 moved these closer to where they're actually used, so it's much easier to understand what they actually do
/// <summary> | ||
/// Subscriber attributes are useful for storing additional, structured information on a user. | ||
/// Since attributes are writable using a public key they should not be used for | ||
/// managing secure or sensitive information such as subscription status, coins, etc. | ||
/// </summary> | ||
/// | ||
/// <remarks>Key names starting with "$" are reserved names used by RevenueCat. For a full list of key | ||
/// restrictions refer [to our guide](https://docs.revenuecat.com/docs/subscriber-attributes) |
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.
most of these docstrings are adapted from iOS
47f1129
to
5c6756d
Compare
5c6756d
to
e2b38a9
Compare
@RevenueCat/core-sdk bump |
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.
RevenueCat/Scripts/Purchases.cs
Outdated
/// </remarks> | ||
/// | ||
/// <param name="products"> The <see cref="StoreProduct"/> for which you want to compute eligibility.</param> | ||
/// <param name="callback"> The <see cref="CheckTrialOrIntroductoryPriceEligibilityCallback"/> callback. </param> |
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.
CheckTrialOrIntroductoryPriceEligibilityCallback
-> CheckTrialOrIntroductoryPriceEligibilityFunc
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.
great catch, thanks!!
Part 2 of adding docstrings to public methods in the Purchases class.
Public members of other public classes don't have docstrings yet, but I might add them later on after having released this.