Skip to content
This repository has been archived by the owner on Jan 24, 2023. It is now read-only.

Fix route schema #2506

Merged
merged 1 commit into from
Jun 25, 2018
Merged

Fix route schema #2506

merged 1 commit into from
Jun 25, 2018

Conversation

richard-cox
Copy link
Contributor

  • entity structure was incorrect (lead to entity.entity.domain)
  • this could have led to some of the app url/domain issue we were seeing in the walkthrough

- entity structure was correct (lead to entity.entity.domain)
- this could have led to some of the app url/domain issue we were seeing in the walkthrough
@cfdreddbot
Copy link

Hey richard-cox!

Thanks for submitting this pull request! I'm here to inform the recipients of the pull request that you and the commit authors have already signed the CLA.

@richard-cox richard-cox changed the title Fix coreRouteSchemaParams Fix route schema Jun 22, 2018
@codecov
Copy link

codecov bot commented Jun 22, 2018

Codecov Report

Merging #2506 into v2-master will decrease coverage by <.01%.
The diff coverage is 100%.

@@              Coverage Diff              @@
##           v2-master    #2506      +/-   ##
=============================================
- Coverage      70.65%   70.64%   -0.01%     
=============================================
  Files            590      590              
  Lines          24822    24821       -1     
  Branches        5611     5611              
=============================================
- Hits           17537    17536       -1     
  Misses          7285     7285

@irfanhabib irfanhabib merged commit 7d73e4a into v2-master Jun 25, 2018
@irfanhabib irfanhabib deleted the fix-domain-schema branch June 25, 2018 13:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants