Skip to content

feat: add support for get, getAll, has and set - #461

Open
char0n wants to merge 1 commit into
form-data:masterfrom
char0n:char0n/additional-methods
Open

char0n wants to merge 1 commit into
form-data:masterfrom
char0n:char0n/additional-methods

Conversation

@char0n

@char0n char0n commented Apr 30, 2020

Copy link
Copy Markdown

Following signatures have not been implemented in this PR:

append(name, blobValue, filename)
set(name, blobValue, filename)

The reason they have not been implemented is to keep the simplicity of first implementation. Those signatures can be added later as additional layer.

To implement this PR I followed the spec at https://xhr.spec.whatwg.org/#create-an-entry.

Let me know if this is something you want inside a library. If I get a green light, I'll implement the tests and in second PR later I'll add the two missing signatures.

Following signatures have not been implemented in this PR:

append(name, blobValue, filename)
set(name, blobValue, filename)
@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage decreased (-4.9%) to 93.305% when pulling d9e67c8 on char0n:char0n/additional-methods into d702625 on form-data:master.

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.

2 participants