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.

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

Rename AppConfiguration#url to baseUrl #3612

Closed
kraenhansen opened this issue Feb 23, 2021 · 0 comments · Fixed by #3727
Closed

Rename AppConfiguration#url to baseUrl #3612

kraenhansen opened this issue Feb 23, 2021 · 0 comments · Fixed by #3727
Assignees
@kraenhansen
Labels
T-Bug

Comments

@kraenhansen
Copy link
Member

kraenhansen commented Feb 23, 2021 •
edited by sync-by-unito bot
Loading

Goals

To align with the other SDKs, our types and Realm Web, I suggest we deprecate the url option in favour of baseUrl on the AppConfiguration:
@kraenhansen kraenhansen changed the title Rename AppConfiguration#url to AppConfiguration#baseUrl Rename AppConfiguration#url to baseUrl Feb 23, 2021
This was referenced May 10, 2021
Merged
Merged
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 16, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@kraenhansen kraenhansen

Labels
T-Bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

"url" → "baseUrl" renamed on App configurations (#3725 on master) realm/realm-js
1 participant
@kraenhansen