-
Notifications
You must be signed in to change notification settings - Fork 13
fix: expose APIFY_USER_IS_PAYING
env var to the configuration
#507
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
Conversation
src/apify/_configuration.py
Outdated
@@ -334,6 +334,14 @@ class Configuration(CrawleeConfiguration): | |||
), | |||
] = None | |||
|
|||
user_is_paying: Annotated[ | |||
str | None, |
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.
Would you be so kind and make it parse into a bool
?
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.
Now it should be parsed into a bool ;)
Add documentation for `APIFY_USER_IS_PAYING` environmental variable. - This is followup of these changes: - apify/apify-sdk-js#415 - apify/apify-shared-python#40 - apify/apify-sdk-python#507 --------- Co-authored-by: stepskop <steps@crashintosh.local> Co-authored-by: stepskop <steps@Mac.localdomain>
APIFY_USER_IS_PAYING
env var annotation to the configurationAPIFY_USER_IS_PAYING
env var to the configuration
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
Adds `APIFY_USER_IS_PAYING` environmental variable annotation to the configuration. - PR with updated documentation is here: apify/apify-docs#1706 - PR with changes to the `apify-shared-python` (adding the env variable to the shared constants): apify/apify-shared-python#40 - (this is a prerequisite, and it needs to be merged first, then the package here needs to be updated as follow-up)
Adds
APIFY_USER_IS_PAYING
environmental variable annotation to the configuration.APIFY_USER_IS_PAYING
env var documentation apify-docs#1706apify-shared-python
(adding the env variable to the shared constants): chore: addAPIFY_USER_IS_PAYING
env var to theApifyEnvVars
apify-shared-python#40 - (this is a prerequisite, and it needs to be merged first, then the package here needs to be updated as follow-up)