Skip to content

v6: PhoneNumber and GeoCoordinates types #489

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.

Sign up for GitHub

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

Jump to bottom
Merged
bevzzz merged 2 commits into from
Oct 28, 2025
Merged

v6: PhoneNumber and GeoCoordinates types #489

bevzzz merged 2 commits into from
Oct 28, 2025

Conversation

@bevzzz
Copy link
Collaborator

@bevzzz bevzzz commented Oct 24, 2025 •
edited
Loading

🧱 Stacked atop #485 (merge that first to get a nicer diff)

This PR adds support for the 2 remaining data types: phoneNumber and geoCoordinates.

The client knows how to serialize them in insert and insertMany as well as unmarshal those from query results. ORM also supports these data types.

PhoneNumber.national("at", "0655 898 86411");
PhoneNumber.international("+43 655 898 86411");

new GeoCoordinates(33.03f, 50.40f);

orca-security-eu[bot]
orca-security-eu bot reviewed Oct 24, 2025
View reviewed changes
Copy link

@orca-security-eu orca-security-eu bot left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca

@bevzzz bevzzz marked this pull request as draft October 24, 2025 11:36
@bevzzz bevzzz self-assigned this Oct 24, 2025
@bevzzz bevzzz force-pushed the v6-geo-phonenumber branch from 20c510e to 94a9e99 Compare October 27, 2025 18:08
@bevzzz bevzzz marked this pull request as ready for review October 27, 2025 18:09
@bevzzz bevzzz merged commit 69d74a6 into v6 Oct 28, 2025
2 of 3 checks passed
@bevzzz bevzzz deleted the v6-geo-phonenumber branch October 28, 2025 12:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@orca-security-eu orca-security-eu[bot] orca-security-eu[bot] left review comments

@dirkkul dirkkul dirkkul approved these changes

Assignees

@bevzzz bevzzz

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@bevzzz @dirkkul