-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
migrate greenhouse to config-based cdk #15344
Merged
Merged
Changes from 15 commits
Commits
Show all changes
57 commits
Select commit
Hold shift + click to select a range
44e9f35
greenhouse minus pagination
girarda 60ab8c3
jobs
girarda ded284c
first substream
girarda 9d9d2f6
Merge branch 'master' into alex/configbasedgreenhouse
girarda f604b76
rename field
girarda 3d86dc8
applications_demographics_answers_stream
girarda 233581b
interviews
girarda df5d5c2
All streams are implemented
girarda 37b7990
fix check
girarda e768dce
fix spec
girarda 12495f2
disable backward compatibility tests
girarda 3cbf8f6
disable backward compatibility tests
girarda 52112ab
unit tests
girarda ca1e20e
definitions
girarda 6cd58f8
only use config.json
girarda 4779738
Merge branch 'master' into alex/configbasedgreenhouse
girarda 3a41750
bump version
girarda 260ec89
expected records
girarda aada550
delete stream classes
girarda a1bb419
Handle extracting no records from root
girarda 677f002
handle missing keys
girarda f049223
Remove unused field from JsonSchema (#15425)
girarda 28d0c7a
:tada: Source File - add support for custom encoding (#15293)
midavadim a3ee499
change query frequency to 1hour (#15499)
xiaohansong 0f780fd
[low-code connectors]: Assert there are no custom top-level fields (#…
girarda 1692b0b
🐞 Postgres source: fix bug in intermediate state emission (#15496)
tuliren b8971c6
🪟 🔧 Add testing and storybook component for CatalogDiffModal (#15426)
teallarson 563870c
:tada: New Source: Hubplanner (#15521)
marcosmarxm ebf8352
Make it possible to specify normalization pod resources. (#15495)
davinchia 4f4a834
[low-code connectors] Extract datetime parser and handle %s format di…
girarda 22a3be4
source-file-secure bump to 0.2.16 (#15528)
brianjlai e3f1166
unit test sendgrid messages stream (#15331)
girarda c5970d3
record extractor interface
girarda 650ef56
dpath extractor
girarda 7024ecc
docstring
girarda e1e4551
🎉 Source File: cache binary stream to file (#15501)
grubberr 2151b77
Docs: update posthog.md (#15541)
juliatournant 6ea5290
Source Stripe: implement slicing (#15292)
davydov-d 2daad7b
fix: revert extraEnv delition in values.yaml for bootloader (#15548)
xpuska513 d78e9b7
SAT: backward compatibility - check that cursor fields were not chang…
alafanechere 6155072
Replace twttr repo to the root build.gradle (#15544)
VitaliiMaltsev c048dbb
Generate separate server endpoints per domain (#15513)
jdpgrailsdev 940bc8c
🐛 Backward compatibility test: Don't fail on updating additionalPrope…
girarda ee8df45
Source Recurly: adds `state_checkpoint_interval` to streams (#13685)
mohamagdy 618bc3d
Merge branch 'master' into alex/configbasedgreenhouse
girarda e3053ba
merge
girarda b6204e8
reset
girarda 347e6ad
use dpath
girarda 362f5ee
Merge branch 'master' into alex/configbasedgreenhouse
girarda bb57372
enable backward compatibility test
girarda b4de8d6
infer types
girarda 3dbf5ed
Revert "infer types"
girarda e07f54b
infer some of the types
girarda 79d8414
some drying
girarda 0269b19
more drying
girarda 1f84dfa
Merge branch 'master' into alex/configbasedgreenhouse
girarda c99a8fc
auto-bump connector version [ci skip]
octavia-squidington-iii File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -4,10 +4,12 @@ connector_image: airbyte/source-greenhouse:dev | |
tests: | ||
spec: | ||
- spec_path: "source_greenhouse/spec.json" | ||
backward_compatibility_tests_config: | ||
disable_for_version: "0.2.7" | ||
connection: | ||
- config_path: "secrets/config.json" | ||
status: "succeed" | ||
- config_path: "secrets/config_users_only.json" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. these credentials don't work, but I confirmed there are |
||
- config_path: "secrets/config.json" | ||
status: "succeed" | ||
- config_path: "integration_tests/config_invalid.json" | ||
status: "failed" | ||
|
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
need to disable this test because the spec is updated