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

[Chore]: Technical: Translate deckgl-layers/cluster-layer #1815

Conversation

HeimEndyd
Copy link
Collaborator

No description provided.

@HeimEndyd HeimEndyd requested a review from heshan0131 May 6, 2022 08:39
import ClusterBuilder, {getGeoJSON} from '../layer-utils/cluster-utils';
import {RGBAColor} from '../../reducers';
import {AggregationLayerProps} from '@deck.gl/aggregation-layers/aggregation-layer';
Copy link
Contributor

@heshan0131 heshan0131 Jun 8, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is this valid import from deck.gl? I think without a path resolver, you would always want to make deck.gl/aggregation-layers export AggregationLayerProps at top level and import from '@deck.gl/aggregation-layers

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's import from npm package, not from deckgl-layers folder. So we can't do anything with the format there

HeimEndyd and others added 3 commits June 17, 2022 17:45
Signed-off-by: Maksim Suslov <maksim.suslov@actionengine.com>
Signed-off-by: Maksim Suslov <maksim.suslov@actionengine.com>
Signed-off-by: Daria Terekhova <daria.terekhova@actionengine.com>
@dariaterekhova-actionengine dariaterekhova-actionengine force-pushed the ms/js_to_ts_convertion/deckgl-layers/cluster-layer branch from 8d67f36 to 8d44846 Compare June 21, 2022 13:59
@delete-merged-branch delete-merged-branch bot deleted the ms/js_to_ts_convertion/deckgl-layers/cluster-layer branch June 23, 2022 15:15
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

3 participants