Skip to content

Add options, multipolygon & types, tree shake, deps - #96

Merged
sheindel merged 26 commits into
mapbox:masterfrom
Agriculture-Intelligence:master
Aug 21, 2023
Merged

Add options, multipolygon & types, tree shake, deps#96
sheindel merged 26 commits into
mapbox:masterfrom
Agriculture-Intelligence:master

Conversation

@charlieforward9

@charlieforward9charlieforward9 commented Aug 18, 2023•
edited
Loading

Copy link
Copy Markdown
Contributor

Rebooting a 7 year stale file converter.
fix#59, fix#68, fix#73 , fix#74, fix#88
close#57, close#84, close#90, close#92
opened #97

charlieforward9and others added 16 commits August 16, 2023 17:02
@charlieforward9
upgrade versions
bc5384d
@charlieforward9
name of ZIP set to folder name
4afcf77
@charlieforward9
Clean Geojson to take the geometries directly.
2c3e5e0
Add support for multipolygon
@charlieforward9
Merge pull request#1from Agriculture-Intelligence:minimal_upgrades
e5bb49f
Minimal_upgrades
@charlieforward9
fix: ERROR: Delete of a bare identifier cannot be used with the "esm"…
e267b8f
… output format due to strict mode
@charlieforward9
Merge pull request#2from Agriculture-Intelligence/minimal_upgrades
b07a2e2
fix: ERROR: Delete of a bare identifier cannot be used with the "esm"…
@charlieforward9
fix: restricting runtime env to browser
440611b
@charlieforward9
Merge pull request#3from Agriculture-Intelligence:minimal_upgrades
5268fc5
Minimal_upgrades
@charlieforward9
fix: designate output type
095eecc
@charlieforward9
Merge pull request#4from Agriculture-Intelligence/minimal_upgrades
abda746
fix: designate output type
@charlieforward9
fix: async await before download
a02e093
@charlieforward9
Merge pull request#5from Agriculture-Intelligence/minimal_upgrades
93f5ab3
fix: async await before download
@charlieforward9
feat: dist folder, with type defs
c99511c
@charlieforward9
Merge pull request#6from Agriculture-Intelligence/minimal_upgrades
7c6063e
feat: dist folder, with type defs
@charlieforward9
fix: explicitly add typings to package
b4f5d34
@charlieforward9
Merge pull request#7from Agriculture-Intelligence/minimal_upgrades
4068f5a
fix: explicitly add typings to package
charlieforward9
charlieforward9 commented Aug 18, 2023
View reviewed changes
Comment threadsrc/zip.js Outdated
@charlieforward9charlieforward9 changed the title deps, #84, #90, typescript declaration file, auto downloaddeps, options: filename, multipolygon, .d.ts, browser only downgrade (easy fix?)Aug 18, 2023
@charlieforward9charlieforward9 mentioned this pull request Aug 19, 2023
Closed
@charlieforward9

charlieforward9 commented Aug 21, 2023•
edited
Loading

Copy link
Copy Markdown
ContributorAuthor

@sheindel

If we can make these changes I feel more confident pulling your PR into the main shp-write repo. Mapbox let me take it over

Good time to wrap this one up too!

@sheindel
sheindel merged commit 4146e58 into mapbox:masterAug 21, 2023
@charlieforward9charlieforward9 changed the title deps, options: filename, multipolygon, .d.ts, browser only downgrade (easy fix?)deps, improved function options, multipolygon, types, depricate download function to browser only functionality.Aug 21, 2023
@charlieforward9charlieforward9 changed the title deps, improved function options, multipolygon, types, depricate download function to browser only functionality.deps, improved function options, multipolygon, types, depricate download api.Aug 21, 2023
@charlieforward9charlieforward9 changed the title deps, improved function options, multipolygon, types, depricate download api.deps, improved function options, multipolygon, types, tree shake.Aug 21, 2023
@charlieforward9charlieforward9 changed the title deps, improved function options, multipolygon, types, tree shake.deps, improved options, add multipolygon & types, tree shake.Aug 21, 2023
@charlieforward9charlieforward9 changed the title deps, improved options, add multipolygon & types, tree shake.Add options, multipolygon & types, tree shake, depsAug 21, 2023
This was referenced Aug 21, 2023
Closed
Closed
Closed
Closed
Closed
@charlieforward9charlieforward9 mentioned this pull request May 23, 2024
Merged
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer
@sheindelsheindelsheindel left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

2 participants

@charlieforward9@sheindel