Skip to content

handle org_account_ids config property as list or json-encoded string - #30

Merged
leslievandemark merged 6 commits into
masterfrom
TDL-26933/handle-org-account-ids-as-json-encoded-strings
Feb 28, 2025
Merged

handle org_account_ids config property as list or json-encoded string#30
leslievandemark merged 6 commits into
masterfrom
TDL-26933/handle-org-account-ids-as-json-encoded-strings

Conversation

@kgerxhi

@kgerxhi kgerxhi commented Feb 24, 2025

Copy link
Copy Markdown
Contributor

Description of change

Allow org_account_ids config property to be input as a list or json-encoded string. Updates the tap to parse the string if present.

Manual QA steps

  • Added unittests

Risks

Rollback steps

  • revert this branch

AI generated code

https://internal.qlik.dev/general/ways-of-working/code-reviews/#guidelines-for-ai-generated-code

  • this PR has been written with the help of GitHub Copilot or another generative AI tool

kgerxhi and others added 3 commits February 24, 2025 20:57
-----------------------------

Co-authored-by: Leslie VanDeMark <leslie.vandemark@qlik.com>
-----------------------------

Co-authored-by: Leslie VanDeMark <leslie.vandemark@qlik.com>
-----------------------------

Co-authored-by: Leslie VanDeMark <leslie.vandemark@qlik.com>
@leslievandemark leslievandemark changed the title Tdl 26933/handle org account ids as json encoded strings handle org_account_ids config property as list or json-encoded string Feb 25, 2025
kgerxhi and others added 3 commits February 26, 2025 20:49
-----------------------------

Co-authored-by: Leslie VanDeMark <leslie.vandemark@qlik.com>
-----------------------------

Co-authored-by: Leslie VanDeMark <leslie.vandemark@qlik.com>
-----------------------------

Co-authored-by: Leslie VanDeMark <leslie.vandemark@qlik.com>
@leslievandemark
leslievandemark merged commit 5a0204c into master Feb 28, 2025
@dsprayberry
dsprayberry deleted the TDL-26933/handle-org-account-ids-as-json-encoded-strings branch July 17, 2025 17:53
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.

2 participants