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

feat/form serialization #1

Merged
merged 23 commits into from
Mar 21, 2024
Merged

feat/form serialization #1

merged 23 commits into from
Mar 21, 2024

Conversation

SilasKenneth
Copy link
Member

  • Initital code for form serialization.
  • Add implementations for different methods.
  • More implementations and tests.
  • Do refactor to urlencode keys and values.

@SilasKenneth SilasKenneth self-assigned this Mar 5, 2024
@SilasKenneth SilasKenneth linked an issue Mar 5, 2024 that may be closed by this pull request
tests/FormSerializationWriterTest.php Show resolved Hide resolved
tests/FormParseNodeTest.php Outdated Show resolved Hide resolved
composer.json Show resolved Hide resolved
src/FormSerializationWriter.php Show resolved Hide resolved
src/FormParseNode.php Outdated Show resolved Hide resolved
.github/CODEOWNERS Outdated Show resolved Hide resolved
Co-authored-by: Vincent Biret <[email protected]>
tests/FormSerializationWriterTest.php Outdated Show resolved Hide resolved
tests/FormSerializationWriterTest.php Outdated Show resolved Hide resolved
Copy link

@SilasKenneth SilasKenneth merged commit 98168e6 into main Mar 21, 2024
10 checks passed
@SilasKenneth SilasKenneth deleted the feat/form-serialization branch March 21, 2024 12:33
@baywet
Copy link
Member

baywet commented Mar 21, 2024

@SilasKenneth can you also PR kiota to:

  1. add an entry to the changelog
  2. add this serialization provider to the default ones in PHP (refiner)
  3. add this dependency in the appsettings under PHP

Thanks!

@SilasKenneth
Copy link
Member Author

Hi @baywet ,the package isn't available of packagist yet, once there I'll add a dependency.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

uri form encoded serialization for PHP
5 participants