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

Entities: Rename baseUrl entities property as baseURL #7983

Merged
merged 1 commit into from
Jul 16, 2018

Conversation

aduth
Copy link
Member

@aduth aduth commented Jul 16, 2018

This pull request seeks to effect new capitalization guidelines around abbreviations (#7670) toward an API freeze. It deprecates the baseUrl property of the core-data module entities in favor of the corrected baseURL naming. I had thought this may have been part of a public API, though I do not think that we expose this currently? (cc @youknowriad)

Testing instructions:

Ensure unit tests pass:

npm run test-unit

Verify no regressions in the retrieval of entities data (unlikely editor would load at all if it was not working).

@aduth aduth added Framework Issues related to broader framework topics, especially as it relates to javascript [Type] Code Quality Issues or PRs that relate to code quality labels Jul 16, 2018
@aduth aduth requested a review from youknowriad July 16, 2018 15:04
Copy link
Contributor

@youknowriad youknowriad left a comment

Choose a reason for hiding this comment

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

yes, it's not exposed 👍

@aduth aduth merged commit 655bce3 into master Jul 16, 2018
@aduth aduth deleted the rename/core-data-base-url branch July 16, 2018 15:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Framework Issues related to broader framework topics, especially as it relates to javascript [Type] Code Quality Issues or PRs that relate to code quality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants