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
[Chore]: Technical: Isolate cloud-providers #1942
[Chore]: Technical: Isolate cloud-providers #1942
Conversation
ec4826c
to
b6354fe
Compare
Signed-off-by: Daria Terekhova <daria.terekhova@actionengine.com>
Make sure to fix import in |
Signed-off-by: Daria Terekhova <daria.terekhova@actionengine.com>
b6354fe
to
ae9307c
Compare
@@ -25,7 +25,8 @@ import {UploadAnimation} from './status-panel'; | |||
import ImageModalContainer from './image-modal-container'; |
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.
unrelated...but saw this duplicated import on line 25 & 27. it can be merged into one line
@@ -21,7 +21,7 @@ | |||
import {OAuthApp} from '@carto/toolkit'; | |||
import Console from 'global/console'; | |||
import CartoIcon from './carto-icon'; | |||
import {Provider} from 'kepler.gl/cloud-providers'; | |||
import {Provider} from '@kepler.gl/cloud-providers'; | |||
import {createDataContainer, formatCsv} from 'kepler.gl/reducers/table-utils'; |
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.
this line needs to be updated too? createDataContainer formatCsv has been moved
Signed-off-by: Daria Terekhova <daria.terekhova@actionengine.com>
0162a23
to
0aee455
Compare
Signed-off-by: Daria Terekhova <daria.terekhova@actionengine.com>
0aee455
to
2db0565
Compare
No description provided.