Skip to content
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

feat(specs): Add MappingInput to Task Inputs #2787

Merged
merged 3 commits into from
Mar 4, 2024

Conversation

jsok
Copy link
Contributor

@jsok jsok commented Feb 26, 2024

🧭 What and Why

New mapping field has been introduced to the BigQuery task input.

🎟 JIRA Ticket: EEX-1012

Changes included:

🧪 Test

Copy link

netlify bot commented Feb 26, 2024

Deploy Preview for api-clients-automation ready!

Name Link
🔨 Latest commit bd17c17
🔍 Latest deploy log https://app.netlify.com/sites/api-clients-automation/deploys/65de78ba94e54900085543f8
😎 Deploy Preview https://deploy-preview-2787--api-clients-automation.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@algolia-bot
Copy link
Collaborator

algolia-bot commented Feb 26, 2024

✗ The generated branch has been deleted.

If the PR has been merged, you can check the generated code on the main branch.
You can still access the code generated on main via this commit.

@jsok jsok force-pushed the eex-1012-mappings-inputs branch 2 times, most recently from 7d4125b to a467c23 Compare February 27, 2024 23:05
@jsok jsok marked this pull request as ready for review February 27, 2024 23:06
@jsok jsok requested a review from a team as a code owner February 27, 2024 23:06
@jsok jsok requested review from damcou and Fluf22 February 27, 2024 23:06
jonathaningram
jonathaningram previously approved these changes Feb 28, 2024
@millotp
Copy link
Collaborator

millotp commented Feb 29, 2024

thanks for keeping the specs up-to-date !

Copy link
Contributor

@Fluf22 Fluf22 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

@benhinchley benhinchley removed their request for review March 4, 2024 22:02
@jsok jsok force-pushed the eex-1012-mappings-inputs branch from 25f4ec3 to 9e63d6c Compare March 4, 2024 22:06
@jsok
Copy link
Contributor Author

jsok commented Mar 4, 2024

@Fluf22 I don't appear to be authorized to merge this PR. Can you please help?

Copy link

github-actions bot commented Mar 4, 2024

@github-actions github-actions bot temporarily deployed to pull request March 4, 2024 22:17 Inactive
Copy link
Member

@shortcuts shortcuts left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

great, thanks!

@shortcuts shortcuts merged commit 2344274 into main Mar 4, 2024
23 checks passed
@shortcuts shortcuts deleted the eex-1012-mappings-inputs branch March 4, 2024 22:45
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.

6 participants