-
Notifications
You must be signed in to change notification settings - Fork 652
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
Remove api_key requirement for data library #3506 #3523
Conversation
Frontend tests were OK 👍 (details) |
@javisantana quick CR? Nacho seems having trouble with connectivity. |
🇪🇸 |
update de app_config.yml.example please |
from meta_dataset, meta_category | ||
where meta_dataset.meta_category_id = meta_category.cartodb_id | ||
query | ||
DATASETS_QUERY = "SELECT * FROM CDB_CommonDataCatalog();" |
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.
I'd list used columns, *
is error prone and doesn't document what will be returned. Since it's going to be processed manually I think this is a must.
Sorry for the delay. I'd really avoid the usage of |
Frontend tests were OK 👍 (details) |
Frontend tests were OK 👍 (details) |
I've taken your comments into acount plus added the removal of the rollbar debug message in c15322d |
Ok, great. Warn about that message removal in #technology channel when it's deployed so people don't get scared. |
Frontend tests were OK 👍 (details) |
Frontend tests were OK 👍 (details) |
Remove api_key requirement for data library #3506
Refer to CartoDB/data-services#137
@juanignaciosl please review