Harshith/test pr 7063 #7382

harshithmullapudi · 2021-10-26T08:43:20Z

What

Describe what the change is solving
It helps to add screenshots if it affects the frontend.

How

Describe the solution

Pre-merge Checklist

Expand the relevant checklist and delete the others.

New Connector

Community member or Airbyter

Airbyter

If this is a community PR, the Airbyte engineer reviewing this PR is responsible for the below items.

Create a non-forked branch based on this PR and test the below items on it
Build is successful
Credentials added to Github CI. Instructions.
/test connector=connectors/<name> command is passing.
New Connector version released on Dockerhub by running the /publish command described here

Updating a connector

Community member or Airbyter

Airbyter

If this is a community PR, the Airbyte engineer reviewing this PR is responsible for the below items.

Create a non-forked branch based on this PR and test the below items on it
Build is successful
Credentials added to Github CI. Instructions.
/test connector=connectors/<name> command is passing.
New Connector version released on Dockerhub by running the /publish command described here

Connector Generator

Issue acceptance criteria met
PR name follows PR naming conventions
If adding a new generator, add it to the list of scaffold modules being tested
The generator test modules (all connectors with -scaffold in their name) have been updated with the latest scaffold by running ./gradlew :airbyte-integrations:connector-templates:generator:testScaffoldTemplates then checking in your changes
Documentation which references the generator is updated as needed.

Add intentory level stream Apply formatting

harshithmullapudi · 2021-10-26T08:43:58Z

/test connector=connectors/source-shopify

🕑 connectors/source-shopify https://github.com/airbytehq/airbyte/actions/runs/1384822309
❌ connectors/source-shopify https://github.com/airbytehq/airbyte/actions/runs/1384822309
🐛 https://gradle.com/s/ff4dzyaxit4eq

Add intentory level stream Apply formatting

harshithmullapudi · 2021-10-26T16:15:37Z

/test connector=connectors/source-shopify

🕑 connectors/source-shopify https://github.com/airbytehq/airbyte/actions/runs/1386521608

…into harshith/test-pr-7063

harshithmullapudi · 2021-10-26T16:41:19Z

/test connector=connectors/source-shopify

🕑 connectors/source-shopify https://github.com/airbytehq/airbyte/actions/runs/1386606557
✅ connectors/source-shopify https://github.com/airbytehq/airbyte/actions/runs/1386606557
Python tests coverage:

	 ---------- coverage: platform linux, python 3.8.10-final-0 -----------
	 Name                                                 Stmts   Miss  Cover
	 ------------------------------------------------------------------------
	 source_acceptance_test/__init__.py                       2      0   100%
	 source_acceptance_test/base.py                          10      4    60%
	 source_acceptance_test/config.py                        74      8    89%
	 source_acceptance_test/conftest.py                     108    108     0%
	 source_acceptance_test/plugin.py                        47     47     0%
	 source_acceptance_test/tests/__init__.py                 4      0   100%
	 source_acceptance_test/tests/test_core.py              200     94    53%
	 source_acceptance_test/tests/test_full_refresh.py       18     11    39%
	 source_acceptance_test/tests/test_incremental.py        69     38    45%
	 source_acceptance_test/utils/__init__.py                 6      0   100%
	 source_acceptance_test/utils/asserts.py                 37      2    95%
	 source_acceptance_test/utils/common.py                  41     24    41%
	 source_acceptance_test/utils/compare.py                 47     20    57%
	 source_acceptance_test/utils/connector_runner.py        82     49    40%
	 source_acceptance_test/utils/json_schema_helper.py     115     14    88%
	 ------------------------------------------------------------------------
	 TOTAL                                                  860    419    51%
	 ---------- coverage: platform linux, python 3.8.10-final-0 -----------
	 Name                          Stmts   Miss  Cover
	 
	 -------------------------------------------------
	   self._warn("No data was collected.", slug="no-data-collected")
	 source_shopify/__init__.py        2      2     0%
	 source_shopify/auth.py           20     20     0%
	 source_shopify/source.py        206    206     0%
	 source_shopify/transform.py      53     53     0%
	 source_shopify/utils.py          57     57     0%
	 -------------------------------------------------
	 TOTAL                           338    338     0%
	 ---------- coverage: platform linux, python 3.8.10-final-0 -----------
	 Name                          Stmts   Miss  Cover
	 -------------------------------------------------
	 source_shopify/__init__.py        2      0   100%
	 source_shopify/auth.py           20     11    45%
	 source_shopify/source.py        206     82    60%
	 source_shopify/transform.py      53      0   100%
	 source_shopify/utils.py          57      6    89%
	 -------------------------------------------------
	 TOTAL                           338     99    71%

harshithmullapudi · 2021-10-27T02:42:15Z

/publish connector=connectors/source-shopify

🕑 connectors/source-shopify https://github.com/airbytehq/airbyte/actions/runs/1388369607
✅ connectors/source-shopify https://github.com/airbytehq/airbyte/actions/runs/1388369607

harshithmullapudi · 2021-10-27T03:45:45Z

/publish connector=connectors/source-shopify

🕑 connectors/source-shopify https://github.com/airbytehq/airbyte/actions/runs/1388514121
❌ connectors/source-shopify https://github.com/airbytehq/airbyte/actions/runs/1388514121

harshithmullapudi · 2021-10-27T03:46:33Z

/test connector=connectors/source-shopify

🕑 connectors/source-shopify https://github.com/airbytehq/airbyte/actions/runs/1388516001
✅ connectors/source-shopify https://github.com/airbytehq/airbyte/actions/runs/1388516001
Python tests coverage:

	 ---------- coverage: platform linux, python 3.8.10-final-0 -----------
	 Name                                                 Stmts   Miss  Cover
	 ------------------------------------------------------------------------
	 source_acceptance_test/__init__.py                       2      0   100%
	 source_acceptance_test/base.py                          10      4    60%
	 source_acceptance_test/config.py                        74      8    89%
	 source_acceptance_test/conftest.py                     108    108     0%
	 source_acceptance_test/plugin.py                        47     47     0%
	 source_acceptance_test/tests/__init__.py                 4      0   100%
	 source_acceptance_test/tests/test_core.py              200     94    53%
	 source_acceptance_test/tests/test_full_refresh.py       18     11    39%
	 source_acceptance_test/tests/test_incremental.py        69     38    45%
	 source_acceptance_test/utils/__init__.py                 6      0   100%
	 source_acceptance_test/utils/asserts.py                 37      2    95%
	 source_acceptance_test/utils/common.py                  41     24    41%
	 source_acceptance_test/utils/compare.py                 47     20    57%
	 source_acceptance_test/utils/connector_runner.py        82     49    40%
	 source_acceptance_test/utils/json_schema_helper.py     115     14    88%
	 ------------------------------------------------------------------------
	 TOTAL                                                  860    419    51%
	 ---------- coverage: platform linux, python 3.8.10-final-0 -----------
	   self._warn("No data was collected.", slug="no-data-collected")
	 Name                          Stmts   Miss  Cover
	 -------------------------------------------------
	 source_shopify/__init__.py        2      2     0%
	 source_shopify/auth.py           20     20     0%
	 source_shopify/source.py        211    211     0%
	 source_shopify/transform.py      53     53     0%
	 source_shopify/utils.py          57     57     0%
	 -------------------------------------------------
	 TOTAL                           343    343     0%
	 ---------- coverage: platform linux, python 3.8.10-final-0 -----------
	 Name                          Stmts   Miss  Cover
	 -------------------------------------------------
	 source_shopify/__init__.py        2      0   100%
	 source_shopify/auth.py           20     11    45%
	 source_shopify/source.py        211     87    59%
	 source_shopify/transform.py      53      0   100%
	 source_shopify/utils.py          57      6    89%
	 -------------------------------------------------
	 TOTAL                           343    104    70%

harshithmullapudi · 2021-10-27T04:36:15Z

/test connector=connectors/source-shopify

🕑 connectors/source-shopify https://github.com/airbytehq/airbyte/actions/runs/1388621589
✅ connectors/source-shopify https://github.com/airbytehq/airbyte/actions/runs/1388621589
Python tests coverage:

	 ---------- coverage: platform linux, python 3.8.10-final-0 -----------
	 Name                                                 Stmts   Miss  Cover
	 ------------------------------------------------------------------------
	 source_acceptance_test/__init__.py                       2      0   100%
	 source_acceptance_test/base.py                          10      4    60%
	 source_acceptance_test/config.py                        74      8    89%
	 source_acceptance_test/conftest.py                     108    108     0%
	 source_acceptance_test/plugin.py                        47     47     0%
	 source_acceptance_test/tests/__init__.py                 4      0   100%
	 source_acceptance_test/tests/test_core.py              200     94    53%
	 source_acceptance_test/tests/test_full_refresh.py       18     11    39%
	 source_acceptance_test/tests/test_incremental.py        69     38    45%
	 source_acceptance_test/utils/__init__.py                 6      0   100%
	 source_acceptance_test/utils/asserts.py                 37      2    95%
	 source_acceptance_test/utils/common.py                  41     24    41%
	 source_acceptance_test/utils/compare.py                 47     20    57%
	 source_acceptance_test/utils/connector_runner.py        82     49    40%
	 source_acceptance_test/utils/json_schema_helper.py     115     14    88%
	 ------------------------------------------------------------------------
	 TOTAL                                                  860    419    51%
	 ---------- coverage: platform linux, python 3.8.10-final-0 -----------
	 Name                          Stmts   Miss  Cover
	 -------------------------------------------------
	 source_shopify/__init__.py        2      2     0%
	 source_shopify/auth.py           20     20     0%
	 source_shopify/source.py        211    211     0%
	 source_shopify/transform.py      53     53     0%
	 source_shopify/utils.py          57     57     0%
	 -------------------------------------------------
	 TOTAL                           343    343     0%
	 ---------- coverage: platform linux, python 3.8.10-final-0 -----------
	 Name                          Stmts   Miss  Cover
	 -------------------------------------------------
	 source_shopify/__init__.py        2      0   100%
	 source_shopify/auth.py           20     11    45%
	 source_shopify/source.py        211     87    59%
	 source_shopify/transform.py      53      0   100%
	 source_shopify/utils.py          57      6    89%
	 -------------------------------------------------
	 TOTAL                           343    104    70%

harshithmullapudi · 2021-10-27T05:23:04Z

/publish connector=connectors/source-shopify

🕑 connectors/source-shopify https://github.com/airbytehq/airbyte/actions/runs/1388731378
✅ connectors/source-shopify https://github.com/airbytehq/airbyte/actions/runs/1388731378

* Add location stream to Shopify connector Add intentory level stream Apply formatting * Update doc and version configs * Update version * Format after rebasing * Address review comments * fix: location stream has no data with the current cred * Add location stream to Shopify connector Add intentory level stream Apply formatting * Update doc and version configs * Update version * Format after rebasing * Address review comments * fix: location stream has no data with the current cred * fix: integration test * fix: integrationtests * fix: integrationtests * Fixed primary key for inventory levels stream Co-authored-by: Yuhui Shi <yuhui@convect.ai>

yuhuishi-convect added 5 commits October 14, 2021 15:12

Add location stream to Shopify connector

249b4c5

Add intentory level stream Apply formatting

Update doc and version configs

7682c48

Update version

d883ac7

Format after rebasing

c2a56ea

Address review comments

4998b86

github-actions bot added area/connectors Connector related issues area/documentation Improvements or additions to documentation labels Oct 26, 2021

harshithmullapudi temporarily deployed to more-secrets October 26, 2021 08:44 Inactive

jrhizor temporarily deployed to more-secrets October 26, 2021 08:46 Inactive

fix: location stream has no data with the current cred

9e5055d

harshithmullapudi temporarily deployed to more-secrets October 26, 2021 16:12 Inactive

yuhuishi-convect and others added 7 commits October 26, 2021 21:43

Add location stream to Shopify connector

7eb2fe6

Add intentory level stream Apply formatting

Update doc and version configs

c327332

Update version

93c8e92

Format after rebasing

0e36fb3

Address review comments

cd55fb9

fix: location stream has no data with the current cred

47777bc

Merge branch 'master' into harshith/test-pr-7063

d5ee3fd

harshithmullapudi temporarily deployed to more-secrets October 26, 2021 16:17 Inactive

jrhizor temporarily deployed to more-secrets October 26, 2021 16:17 Inactive

harshithmullapudi added 2 commits October 26, 2021 22:09

Merge branch 'harshith/test-pr-7063' of github.com:airbytehq/airbyte …

2d75aac

…into harshith/test-pr-7063

fix: integration test

0f4b62b

harshithmullapudi temporarily deployed to more-secrets October 26, 2021 16:42 Inactive

jrhizor temporarily deployed to more-secrets October 26, 2021 16:43 Inactive

jrhizor temporarily deployed to more-secrets October 27, 2021 02:44 Inactive

fix: integrationtests

ffba5ba

fix: integrationtests

6ba31b6

harshithmullapudi temporarily deployed to more-secrets October 27, 2021 03:47 Inactive

jrhizor temporarily deployed to more-secrets October 27, 2021 03:48 Inactive

harshithmullapudi temporarily deployed to more-secrets October 27, 2021 04:33 Inactive

Fixed primary key for inventory levels stream

bddad56

harshithmullapudi force-pushed the harshith/test-pr-7063 branch from 04f401d to bddad56 Compare October 27, 2021 04:36

harshithmullapudi temporarily deployed to more-secrets October 27, 2021 04:38 Inactive

jrhizor temporarily deployed to more-secrets October 27, 2021 04:38 Inactive

harshithmullapudi mentioned this pull request Oct 27, 2021

🎉 Source Shopify: Add InventoryLevels and Locations streams #7063

Closed

38 tasks

jrhizor temporarily deployed to more-secrets October 27, 2021 05:25 Inactive

harshithmullapudi merged commit e479388 into master Oct 27, 2021

harshithmullapudi deleted the harshith/test-pr-7063 branch October 27, 2021 06:05

jrhizor mentioned this pull request Nov 1, 2021

Bump Airbyte version from 0.30.23-alpha to 0.30.24-alpha #7532

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Harshith/test pr 7063 #7382

Harshith/test pr 7063 #7382

harshithmullapudi commented Oct 26, 2021

harshithmullapudi commented Oct 26, 2021 •

edited by github-actions bot

Loading

harshithmullapudi commented Oct 26, 2021 •

edited by github-actions bot

Loading

harshithmullapudi commented Oct 26, 2021 •

edited by github-actions bot

Loading

harshithmullapudi commented Oct 27, 2021 •

edited by github-actions bot

Loading

harshithmullapudi commented Oct 27, 2021 •

edited by github-actions bot

Loading

harshithmullapudi commented Oct 27, 2021 •

edited by github-actions bot

Loading

harshithmullapudi commented Oct 27, 2021 •

edited by github-actions bot

Loading

harshithmullapudi commented Oct 27, 2021 •

edited by github-actions bot

Loading

Harshith/test pr 7063 #7382

Harshith/test pr 7063 #7382

Conversation

harshithmullapudi commented Oct 26, 2021

What

How

Recommended reading order

Pre-merge Checklist

Community member or Airbyter

Airbyter

Community member or Airbyter

Airbyter

harshithmullapudi commented Oct 26, 2021 • edited by github-actions bot Loading

harshithmullapudi commented Oct 26, 2021 • edited by github-actions bot Loading

harshithmullapudi commented Oct 26, 2021 • edited by github-actions bot Loading

harshithmullapudi commented Oct 27, 2021 • edited by github-actions bot Loading

harshithmullapudi commented Oct 27, 2021 • edited by github-actions bot Loading

harshithmullapudi commented Oct 27, 2021 • edited by github-actions bot Loading

harshithmullapudi commented Oct 27, 2021 • edited by github-actions bot Loading

harshithmullapudi commented Oct 27, 2021 • edited by github-actions bot Loading

harshithmullapudi commented Oct 26, 2021 •

edited by github-actions bot

Loading

harshithmullapudi commented Oct 26, 2021 •

edited by github-actions bot

Loading

harshithmullapudi commented Oct 26, 2021 •

edited by github-actions bot

Loading

harshithmullapudi commented Oct 27, 2021 •

edited by github-actions bot

Loading

harshithmullapudi commented Oct 27, 2021 •

edited by github-actions bot

Loading

harshithmullapudi commented Oct 27, 2021 •

edited by github-actions bot

Loading

harshithmullapudi commented Oct 27, 2021 •

edited by github-actions bot

Loading

harshithmullapudi commented Oct 27, 2021 •

edited by github-actions bot

Loading