-
Notifications
You must be signed in to change notification settings - Fork 105
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
bug/1382_carto_enterprise_individual_accounts #1393
Conversation
frbattid
commented
Jan 27, 2017
•
edited
edited
- Fixes issue [cygnus-ngsi][NGSICartoSink] Differentiate among enterprise and individual accounts #1382
- e2e tests done by @emepetres, who owns a personal Carto account with API key :)
…t type in carto keys file
… on the account type
@@ -8,3 +8,4 @@ | |||
- [cygnus-ngsi][hardening] Add raw bytes to NGSIEvent (#1345) | |||
- [cygnus-ngsi][hardening] Add cache to NGSIMySQLSink (#130) | |||
- [cygnus][hardening] Fix "FIWARE" name in license headers (#1369) | |||
- [cygnus-ngsi][bug] Differentiate among enterprise and individual accounts (#1382) |
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.
Maybe better
[cygnus-ngsi][bug] Differentiate among enterprise and individual Carto accounts (#1382)
?
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.
Fixed in f2d7e43
@@ -31,6 +31,7 @@ | |||
public class CartoDBBackendImpl extends HttpBackend implements CartoDBBackend { | |||
|
|||
private final String apiKey; | |||
private final boolean isPersonal; |
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.
isPersonal -> isPersonalAccount ?
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.
Fixed in f2d7e43
@@ -39,17 +40,23 @@ | |||
* @param port | |||
* @param ssl | |||
* @param apiKey | |||
* @param isPersonal |
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.
isPersonal -> isPersonalAccount
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.
Fixed in 642cc87
LGTM |