Skip to content

Conversation

@Amruta-Ranade
Copy link
Contributor

Edited UI strings for the Google Sheets and Salesforce source connectors and Snowflake destination connector.

My first time editing strings in the code -- let me know if I did it right!

@codecov
Copy link

codecov bot commented Jun 13, 2022

Codecov Report

❗ No coverage uploaded for pull request base (master@dcf4650). Click here to learn what that means.
The diff coverage is n/a.

❗ Current head 736c05d differs from pull request most recent head 8065b00. Consider uploading reports for the commit 8065b00 to get more accurate results

@@ Coverage Diff @@ ## master #13729 +/- ## ========================================= Coverage ? 79.48% ========================================= Files ? 14 Lines ? 892 Branches ? 0 ========================================= Hits ? 709 Misses ? 183 Partials ? 0 

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dcf4650...8065b00. Read the comment docs.

@sherifnada
Copy link
Contributor

@airbytehq/api-connectors-dx

@sherifnada
Copy link
Contributor

sherifnada commented Jun 13, 2022

/publish connector=connectors/source-salesforce

🕑 connectors/source-salesforce https://github.com/airbytehq/airbyte/actions/runs/2491162415
🚀 Successfully published connectors/source-salesforce
🚀 Auto-bumped version for connectors/source-salesforce
✅ connectors/source-salesforce https://github.com/airbytehq/airbyte/actions/runs/2491162415

@sherifnada
Copy link
Contributor

sherifnada commented Jun 13, 2022

/publish connector=connectors/destination-snowflake

🕑 connectors/destination-snowflake https://github.com/airbytehq/airbyte/actions/runs/2491162794
❌ Failed to publish connectors/destination-snowflake
❌ Couldn't auto-bump version for connectors/destination-snowflake

@sherifnada
Copy link
Contributor

sherifnada commented Jun 13, 2022

/publish connector=connectors/source-google-sheets

🕑 connectors/source-google-sheets https://github.com/airbytehq/airbyte/actions/runs/2491164582
❌ Failed to publish connectors/source-google-sheets
❌ Couldn't auto-bump version for connectors/source-google-sheets

ENTRYPOINT ["python", "/airbyte/integration_code/main.py"]

LABEL io.airbyte.version=1.0.9
LABEL io.airbyte.version=1.1.0
Copy link
Contributor

Choose a reason for hiding this comment

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

Note: It's ok to say 1.0.10 - unsure if bumping the minor was intended

cc @sherifnada

Copy link
Contributor

Choose a reason for hiding this comment

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

+1. this doesn't have to be a minor release

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh, my bad. How do I fix this? Pushing a commit gives me the following error:

error: failed to push some refs to 'github.com:airbytehq/airbyte.git'
hint: Updates were rejected because the remote contains work that you do
hint: not have locally. This is usually caused by another repository pushing
hint: to the same ref. You may want to first integrate the remote changes
hint: (e.g., 'git pull ...') before pushing again.
hint: See the 'Note about fast-forwards' in 'git push --help' for details.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The obvious answer seems to be git pull then git push but want to make sure that's correct

Copy link
Contributor Author

@Amruta-Ranade Amruta-Ranade Jun 14, 2022

Choose a reason for hiding this comment

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

Okay, I changed the version in the dockerfile. How do I update the source_specs and source_definitions files?
Update: I see the merge conflicts but not sure if I should resolve them manually or if our automation workflow takes care of them

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think I fixed it. Can someone verify and tell me what to do next?

@octavia-squidington-iii octavia-squidington-iii temporarily deployed to more-secrets June 13, 2022 21:27 Inactive
@Amruta-Ranade Amruta-Ranade temporarily deployed to more-secrets June 14, 2022 14:47 Inactive
@Amruta-Ranade Amruta-Ranade temporarily deployed to more-secrets June 14, 2022 18:27 Inactive
@sherifnada
Copy link
Contributor

sherifnada commented Jun 16, 2022

/publish connector=connectors/destination-snowflake

🕑 connectors/destination-snowflake https://github.com/airbytehq/airbyte/actions/runs/2512334723
❌ Failed to publish connectors/destination-snowflake
❌ Couldn't auto-bump version for connectors/destination-snowflake

@sherifnada
Copy link
Contributor

sherifnada commented Jun 16, 2022

/publish connector=connectors/source-google-sheets

🕑 connectors/source-google-sheets https://github.com/airbytehq/airbyte/actions/runs/2512335323
❌ Failed to publish connectors/source-google-sheets
❌ Couldn't auto-bump version for connectors/source-google-sheets

@Amruta-Ranade Amruta-Ranade temporarily deployed to more-secrets July 6, 2022 21:23 Inactive
@sherifnada sherifnada requested a review from alafanechere July 7, 2022 00:50
@alafanechere alafanechere self-assigned this Jul 7, 2022
@alafanechere alafanechere temporarily deployed to more-secrets July 7, 2022 08:48 Inactive
@github-actions github-actions bot added the area/documentation Improvements or additions to documentation label Jul 7, 2022
@alafanechere alafanechere temporarily deployed to more-secrets July 7, 2022 09:07 Inactive
@alafanechere
Copy link
Contributor

alafanechere commented Jul 7, 2022

/publish connector=connectors/destination-snowlake,connectors/source-salesforce

🕑 Publishing the following connectors:
connectors/destination-snowlake,connectors/source-salesforce
https://github.com/airbytehq/airbyte/actions/runs/2628467661


Connector Did it publish? Were definitions generated?
connectors/source-salesforce
connectors/destination-snowlake

if you have connectors that successfully published but failed definition generation, follow step 4 here ▶️

@alafanechere
Copy link
Contributor

alafanechere commented Jul 7, 2022

/publish connector=connectors/destination-snowlake,connectors/source-salesforce

🕑 Publishing the following connectors:
connectors/destination-snowlake,connectors/source-salesforce
https://github.com/airbytehq/airbyte/actions/runs/2628467943


Connector Did it publish? Were definitions generated?
connectors/source-salesforce
connectors/destination-snowlake

if you have connectors that successfully published but failed definition generation, follow step 4 here ▶️

@alafanechere
Copy link
Contributor

alafanechere commented Jul 7, 2022

/publish connector=connectors/destination-snowflake

🕑 Publishing the following connectors:
connectors/destination-snowflake
https://github.com/airbytehq/airbyte/actions/runs/2628488433


Connector Did it publish? Were definitions generated?
connectors/destination-snowflake

if you have connectors that successfully published but failed definition generation, follow step 4 here ▶️

@octavia-squidington-iii octavia-squidington-iii temporarily deployed to more-secrets July 7, 2022 09:36 Inactive
@octavia-squidington-iii octavia-squidington-iii temporarily deployed to more-secrets July 7, 2022 10:20 Inactive
@alafanechere
Copy link
Contributor

alafanechere commented Jul 7, 2022

/publish connector=connectors/source-google-sheets

🕑 Publishing the following connectors:
connectors/source-google-sheets
https://github.com/airbytehq/airbyte/actions/runs/2628897531


Connector Did it publish? Were definitions generated?
connectors/source-google-sheets

if you have connectors that successfully published but failed definition generation, follow step 4 here ▶️

@alafanechere alafanechere temporarily deployed to more-secrets July 7, 2022 12:56 Inactive
@alafanechere
Copy link
Contributor

alafanechere commented Jul 7, 2022

/publish connector=connectors/source-google-sheets

🕑 Publishing the following connectors:
connectors/source-google-sheets
https://github.com/airbytehq/airbyte/actions/runs/2629876385


Connector Did it publish? Were definitions generated?
connectors/source-google-sheets

if you have connectors that successfully published but failed definition generation, follow step 4 here ▶️

@alafanechere alafanechere temporarily deployed to more-secrets July 7, 2022 13:30 Inactive
@octavia-squidington-iii octavia-squidington-iii temporarily deployed to more-secrets July 7, 2022 13:48 Inactive
Copy link
Contributor

@alafanechere alafanechere left a comment

Choose a reason for hiding this comment

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

After fixing a quote issue in the google-sheet connector I was able to successfully publish a new version for the three connectors:

  • source-google-sheets
  • source-salesforce
  • destination-snowflake

I'll merge now.

@alafanechere alafanechere merged commit d4071a0 into master Jul 7, 2022
@alafanechere alafanechere deleted the edit_GA_connector_UI_strings branch July 7, 2022 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment