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

REFACTOR LocationPage modeling #223

Merged
merged 14 commits into from
Jul 21, 2020
Merged

REFACTOR LocationPage modeling #223

merged 14 commits into from
Jul 21, 2020

Conversation

muskie9
Copy link
Member

@muskie9 muskie9 commented Mar 19, 2020

Update the overall model pattern to be Page based.

muskie9 added 2 commits March 19, 2020 14:31
Update the overall model pattern to be Page based.
_config/config.yml Outdated Show resolved Hide resolved
@codecov
Copy link

codecov bot commented Mar 19, 2020

Codecov Report

Merging #223 into master will decrease coverage by 15.99%.
The diff coverage is 12.62%.

Impacted file tree graph

@@              Coverage Diff              @@
##             master     #223       +/-   ##
=============================================
- Coverage     61.89%   45.89%   -16.00%     
- Complexity      105      161       +56     
=============================================
  Files             7       12        +5     
  Lines           370      512      +142     
=============================================
+ Hits            229      235        +6     
- Misses          141      277      +136     
Flag Coverage Δ Complexity Δ
#php 45.89% <12.62%> (-16.00%) 161.00 <34.00> (+56.00) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ Complexity Δ
src/Model/Location.php 73.58% <ø> (ø) 16.00 <0.00> (?)
src/Page/LocationPage.php 0.00% <0.00%> (ø) 16.00 <16.00> (?)
src/Tasks/EmailAddressTask.php 0.00% <0.00%> (ø) 6.00 <0.00> (?)
src/Tasks/LocationCategoriesTask.php 0.00% <ø> (ø) 9.00 <0.00> (?)
src/Tasks/LocationPublishTask.php 0.00% <ø> (ø) 14.00 <0.00> (?)
src/Tasks/LocationToPageTask.php 0.00% <0.00%> (ø) 11.00 <11.00> (?)
src/admin/LocationAdmin.php 0.00% <0.00%> (ø) 6.00 <0.00> (+1.00)
src/bulkloader/LocationCsvBulkLoader.php 0.00% <ø> (ø) 9.00 <0.00> (ø)
src/form/LocatorForm.php 97.67% <ø> (+16.27%) 11.00 <0.00> (ø)
src/Page/Locator.php 87.14% <87.50%> (ø) 23.00 <5.00> (?)
... and 9 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4e09b0a...b6c65e1. Read the comment docs.

@mak001
Copy link
Contributor

mak001 commented Jul 16, 2020

since this will be used in a new major release, are there any objections to implementing #187

@muskie9
Copy link
Member Author

muskie9 commented Jul 16, 2020

@mak001 good idea, let's make sure we get that tackled before releasing the new major.

jsirish added 3 commits July 20, 2020 20:05
LocationPage
* ShowInMenus default false
* Moved contact info to CMS tab
* getWebsiteURL() - add check for http/s
@jsirish jsirish merged commit c54da72 into dynamic:master Jul 21, 2020
@jsirish jsirish deleted the pull/v4 branch July 21, 2020 19:48
mak001 pushed a commit that referenced this pull request Jul 28, 2022
Update the overall model pattern to be Page based.
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.

3 participants