CartoDB / cartodb

Location Intelligence & Data Visualization tool
http://carto.com
BSD 3-Clause "New" or "Revised" License
2.75k stars 650 forks source link

Mobile api keys design review #9005

Closed javitonino closed 8 years ago

javitonino commented 8 years ago

@javitonino commented on Fri Jul 15 2016

Hey @josecruz: After meeting with @jaakla to prepare the launch of the CartoSDK, he proposed some changes to the design. Let's meet when you are available and I'll give you the details, please :pray:


@jaakla commented on Fri Jul 15 2016

Also please remove Carto apiKey from this screen, as we cannot use it in mobile at all due to security reasons.


@josecruz commented on Wed Jul 20 2016

Hey! I upload here last changes: api keys - 00 - 02 - edit form

javitonino commented 8 years ago

A bit confusing, but moved the issue to this repo where it made more sense. We want to have this ready for this milestone. Basically:

@MariaCheca This implies a minor controller change in the backend, ping me if you need help to do the change, or you prefer for me to do it.

jaakla commented 8 years ago

Suggestion on design - do not use term "API Key" in this page at all, otherwise it would be confused by general CARTO API Key, which is different value and cannot be used in mobile. Mobile SDK has only "License Key".

saleiva commented 8 years ago

+1 to that @jaakla cc/ @josecruz

josecruz commented 8 years ago

Cool, updated that on the first comment!