Skip to content
This repository has been archived by the owner on Oct 7, 2024. It is now read-only.

Admin role and dashboard #27

Merged
merged 1 commit into from
Sep 5, 2018
Merged

Admin role and dashboard #27

merged 1 commit into from
Sep 5, 2018

Conversation

lislis
Copy link
Member

@lislis lislis commented Sep 3, 2018

No description provided.

@coveralls
Copy link

coveralls commented Sep 3, 2018

Pull Request Test Coverage Report for Build 88

  • 204 of 206 (99.03%) changed or added relevant lines in 17 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+1.8%) to 97.699%

Changes Missing Coverage Covered Lines Changed/Added Lines %
app/controllers/users_controller.rb 22 23 95.65%
spec/features/admin_make_admin_spec.rb 14 15 93.33%
Totals Coverage Status
Change from base Build 51: 1.8%
Covered Lines: 1189
Relevant Lines: 1217

💛 - Coveralls

basic admin functionality

add translation strings

prep tests

add some feature test ideas

add user controller test

admon controller tests

add admin rights test

admin delete user feature test

admin delete user feature test

make admin feature test

set test to pending
@lislis lislis changed the title [WIP] Admin role Admin role and dashboard Sep 5, 2018
@lislis lislis merged commit d93682a into master Sep 5, 2018
@lislis lislis deleted the admin-role branch September 5, 2018 14:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants