Skip to content

New admin - Settings section#1115

Merged
beagleknight merged 6 commits intorevamp_adminfrom
new-admin/settings
Mar 15, 2017
Merged

New admin - Settings section#1115
beagleknight merged 6 commits intorevamp_adminfrom
new-admin/settings

Conversation

@beagleknight
Copy link
Copy Markdown
Contributor

@beagleknight beagleknight commented Mar 14, 2017

🎩 What? Why?

Add settings and scopes admin sections to the new admin.

📌 Related Issues

📋 Subtasks

  • Basic settings layout
  • Settings
  • Scopes

📷 Screenshots (optional)

image
image

👻 GIF

None

@mention-bot
Copy link
Copy Markdown

@beagleknight, thanks for your PR! By analyzing the history of the files in this pull request, we identified @mrcasals and @josepjaume to be potential reviewers.

</div>

<div class="columns xlarge-6">
<div class="label--tabs">
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nice! Are they just foundation tabs?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yeah, they are foundation tabs :)

@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #1115 into revamp_admin will increase coverage by 0.12%.
The diff coverage is 100%.

@@               Coverage Diff                @@
##           revamp_admin    #1115      +/-   ##
================================================
+ Coverage         96.99%   97.12%   +0.12%     
================================================
  Files               413      412       -1     
  Lines              6916     6876      -40     
================================================
- Hits               6708     6678      -30     
+ Misses              208      198      -10
Impacted Files Coverage Δ
...ntrollers/decidim/admin/organization_controller.rb 91.3% <100%> (+0.39%)
decidim-core/lib/decidim/form_builder.rb 97.81% <100%> (+0.03%)
decidim-core/app/models/decidim/organization.rb 94.44% <100%> (+0.32%)
...in/app/helpers/decidim/admin/application_helper.rb 100% <100%> (ø)
...m-admin/app/commands/decidim/admin/update_scope.rb 75% <0%> (-25%)
...m-admin/app/commands/decidim/admin/create_scope.rb 76.92% <0%> (-23.08%)
...ents/app/commands/decidim/comments/vote_comment.rb 100% <0%> (+13.33%)
...cidim-comments/lib/decidim/comments/commentable.rb 100% <0%> (+17.64%)
...ts/app/queries/decidim/comments/sorted_comments.rb 97.05% <0%> (+23.52%)

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 5e53f44...5fc8f76. Read the comment docs.

@codecov-io
Copy link
Copy Markdown

codecov-io commented Mar 15, 2017

Codecov Report

Merging #1115 into revamp_admin will increase coverage by 0.19%.
The diff coverage is 100%.

@@               Coverage Diff                @@
##           revamp_admin    #1115      +/-   ##
================================================
+ Coverage         96.99%   97.18%   +0.19%     
================================================
  Files               413      413              
  Lines              6916     6921       +5     
================================================
+ Hits               6708     6726      +18     
+ Misses              208      195      -13
Impacted Files Coverage Δ
...ntrollers/decidim/admin/organization_controller.rb 91.3% <100%> (+0.39%)
decidim-core/lib/decidim/form_builder.rb 97.81% <100%> (+0.03%)
...in/app/helpers/decidim/admin/application_helper.rb 100% <100%> (ø)
decidim-core/app/models/decidim/organization.rb 94.44% <100%> (+0.32%)
...app/controllers/decidim/admin/scopes_controller.rb 91.11% <100%> (+0.2%)
...ents/app/commands/decidim/comments/vote_comment.rb 100% <0%> (+13.33%)
...cidim-comments/lib/decidim/comments/commentable.rb 100% <0%> (+17.64%)
...ts/app/queries/decidim/comments/sorted_comments.rb 97.05% <0%> (+23.52%)

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 5e53f44...5fc8f76. Read the comment docs.

@beagleknight beagleknight merged commit 0610cb0 into revamp_admin Mar 15, 2017
@beagleknight beagleknight deleted the new-admin/settings branch March 15, 2017 09:29
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.

4 participants