Skip to content

Comments

Allow and prefer non-prefixed extra fields for SalesforceHook#27075

Merged
dstandish merged 2 commits intoapache:mainfrom
astronomer:salesforce-remove-extra-prefix
Oct 22, 2022
Merged

Allow and prefer non-prefixed extra fields for SalesforceHook#27075
dstandish merged 2 commits intoapache:mainfrom
astronomer:salesforce-remove-extra-prefix

Conversation

@dstandish
Copy link
Contributor

From airflow version 2.3, extra prefixes are not required so we enable them here.

From airflow version 2.3, extra prefixes are not required so we enable them here.
@Taragolis
Copy link
Contributor

@dstandish seems like another tests in test_credentials_provider might failed due to additional log records.
And it is again asyncio: Task was destroyed but it is pending!

@dstandish dstandish merged commit eaeab80 into apache:main Oct 22, 2022
@dstandish dstandish deleted the salesforce-remove-extra-prefix branch October 22, 2022 19:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Salesforce Provider: connection params in extra require "extra__salesforce__" prefix

3 participants