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

Stabalize the PHP SDK #11

Merged
merged 42 commits into from
Nov 5, 2024
Merged

Conversation

welpie21
Copy link
Contributor

@welpie21 welpie21 commented Oct 8, 2024

Thank you for submitting this pull request! We appreciate you spending the time to work on these changes.

What is the motivation?

To make sure the PHP SDK is up-to-date with all the latest changes of Surreal V2 and that it is stabalized.

What does this change do?

  • added Future, Range, RecordIdRange, BoundIncluded and BoundExcluded types
  • added more functionality to the Duration class
  • added new methods rpc, insertRelation and upsert
  • added more unit tests
  • Updated dependencies
  • Updated readme
  • changes to the unit testing setup

Fixed

  • Fix CBOR encoder map for native DateTime related classes

Breaking

  • renamed disconnect to close in the Surreal Class
  • Moved record related classes under namespace Surreal\Cbor\Types\Record

What is your testing strategy?

I added unit tests to the PHP SDK for versions v1.4.2, v1.5.4 and 2.0.4. Currently we have code coverage of roughly 48% ( rounded down ).

Is this related to any issues?

No

Have you read the Contributing Guidelines?

welpie21 added 30 commits July 21, 2024 13:40
- Meet the same functionalities of the JS SDK ( ported over ).
- Improved API representation by adding `toString()` to all cbor data classes
- Some CBOR data classes having getting additional functionalities reflected based on JS SDK.
- Improved function documentation
- exposed rpc method
- added upsert method
- compatibility changes for auth methods ( wip )
@welpie21 welpie21 marked this pull request as ready for review October 28, 2024 17:07
@kearfy kearfy merged commit 8083292 into surrealdb:main Nov 5, 2024
2 checks passed
@macjuul macjuul mentioned this pull request Nov 6, 2024
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