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

[Enhancement] set initial layer config when set layer type #1898

Conversation

jwasilgeo
Copy link
Collaborator

@jwasilgeo jwasilgeo commented Aug 3, 2022

Signed-off-by: Shan He heshan0131@gmail.com

[Bug] fix setting layer type override layer name

Signed-off-by: Shan He heshan0131@gmail.com
Signed-off-by: Jacob Wasilkowski jwasilkowski@foursquare.com

@jwasilgeo jwasilgeo force-pushed the jw/upstream-enhancement-set-initial-layer-config-when-set-layer-type-43466a9 branch from c1d0d19 to 39f17e3 Compare August 3, 2022 16:20
@jwasilgeo jwasilgeo marked this pull request as ready for review August 3, 2022 16:27
@heshan0131
Copy link
Contributor

please make sure to remove (UN-1863) from the final squashed commit message

src/layers/src/h3-hexagon-layer/h3-hexagon-layer.ts Outdated Show resolved Hide resolved
src/layers/src/point-layer/point-layer.ts Outdated Show resolved Hide resolved
@heshan0131
Copy link
Contributor

Same here...remove UN-1863 from squashed message

@jwasilgeo jwasilgeo force-pushed the jw/upstream-enhancement-set-initial-layer-config-when-set-layer-type-43466a9 branch from 39f17e3 to 9c9bd1b Compare August 4, 2022 01:32
heshan0131 and others added 2 commits August 3, 2022 21:33
Signed-off-by: Shan He <heshan0131@gmail.com>

[Bug] fix setting layer type override layer name

Signed-off-by: Shan He <heshan0131@gmail.com>
Signed-off-by: Jacob Wasilkowski <jwasilkowski@foursquare.com>
Signed-off-by: Jacob Wasilkowski <jwasilkowski@foursquare.com>
@jwasilgeo jwasilgeo force-pushed the jw/upstream-enhancement-set-initial-layer-config-when-set-layer-type-43466a9 branch from 9c9bd1b to d53ea5e Compare August 4, 2022 01:34
@jwasilgeo
Copy link
Collaborator Author

Good catch, I cleaned that up @heshan0131, thanks.

Signed-off-by: Jacob Wasilkowski <jwasilkowski@foursquare.com>
@jwasilgeo jwasilgeo merged commit 9d171c6 into master Aug 4, 2022
@delete-merged-branch delete-merged-branch bot deleted the jw/upstream-enhancement-set-initial-layer-config-when-set-layer-type-43466a9 branch August 4, 2022 01:58
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