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

Ensure User is passed to all Leaf pages #19

Merged
merged 7 commits into from
Aug 27, 2017
Merged

Ensure User is passed to all Leaf pages #19

merged 7 commits into from
Aug 27, 2017

Conversation

0xTim
Copy link
Member

@0xTim 0xTim commented Aug 27, 2017

This should simplify some of the Leaf views by ensuring there is always a user passed to a page if ones is logged in

  • Admin page
  • Create post page
  • Edit post page
  • Create user page
  • Edit user page
  • Reset password page
  • README

@codecov
Copy link

codecov bot commented Aug 27, 2017

Codecov Report

Merging #19 into master will increase coverage by 0.04%.
The diff coverage is 89.18%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master     #19      +/-   ##
=========================================
+ Coverage   77.25%   77.3%   +0.04%     
=========================================
  Files          14      14              
  Lines        1442    1445       +3     
=========================================
+ Hits         1114    1117       +3     
  Misses        328     328
Impacted Files Coverage Δ
Sources/SteamPress/Views/LeafViewFactory.swift 98.72% <100%> (+0.01%) ⬆️
...ources/SteamPress/Controllers/BlogController.swift 83.33% <100%> (ø) ⬆️
...s/SteamPress/Controllers/BlogAdminController.swift 84.48% <76.47%> (ø) ⬆️

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 be113b7...e8f327c. Read the comment docs.

@0xTim 0xTim merged commit 7281079 into master Aug 27, 2017
@0xTim 0xTim deleted the userAdminLeafViews branch August 27, 2017 11:40
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.

None yet

1 participant