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

Staging -> Levelbuilder #30367

Merged
merged 37 commits into from Aug 22, 2019
Merged

Staging -> Levelbuilder #30367

merged 37 commits into from Aug 22, 2019

Conversation

wjordan
Copy link
Contributor

@wjordan wjordan commented Aug 22, 2019

Levelbuilder-hotfix to fix some application issues related to secrets integration.

Erin Peach and others added 30 commits August 9, 2019 10:58
Allow keys such as dashboard_db_reader to depend on db_reader secret.
add test_erb_string_secret
Right now this shows ALL tables in the shared firebase channel, not just ones the user has chosen to "import"
- de-dupe some code in applab
- Make getColumnsRef() db agnostic so it can be used to update column names for project and shared tables
…onfirmation-email

Skip click tracking for confirmation emails.
Flash Firmware on the Circuit Playground Classic
DTS (Levelbuilder > Staging) [robo-dts]
Workaround for running unit test in non-secret environments.
[ci skip]
add google_maps_api_key to unit test config
wjordan and others added 7 commits August 21, 2019 14:35
This unit test currently requires a secret key, which causes it to
fail in test runs where secrets are not provided.
The test should be updated to use a fake/fixture key, or stub out
CloudFront cookie-signing API to remove the dependency.
[ci skip]
…gner-test

skip ApiControllerTest#sign_cookies test
[Applab Datasets] Add applabDatasets experiment and show "current tables" in data tab
Levelbuilder requires secrets for devise and secret_key_base config.
[ci skip]
@wjordan wjordan merged commit 2a0200d into levelbuilder Aug 22, 2019
@wjordan wjordan deleted the levelbuilder-dts branch August 28, 2019 18:24
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.

None yet

5 participants