Skip to content

[New Admin] Update process admins views.#1183

Merged
lastpotion merged 8 commits intorevamp_adminfrom
update/admin_processes
Mar 28, 2017
Merged

[New Admin] Update process admins views.#1183
lastpotion merged 8 commits intorevamp_adminfrom
update/admin_processes

Conversation

@lastpotion
Copy link
Copy Markdown
Contributor

@lastpotion lastpotion commented Mar 27, 2017

🎩 What? Why?

Update process admin section with the new admin views and styles.

📌 Related Issues

📋 Subtasks

  • Update views
  • Fix tests

📷 Screenshots (optional)

screen shot 2017-03-27 at 14 53 36
screen shot 2017-03-27 at 14 53 43
screen shot 2017-03-27 at 14 53 58
screen shot 2017-03-27 at 15 32 46

👻 GIF

@josepjaume josepjaume mentioned this pull request Mar 27, 2017
28 tasks
@codecov-io
Copy link
Copy Markdown

codecov-io commented Mar 27, 2017

Codecov Report

Merging #1183 into revamp_admin will decrease coverage by 0.05%.
The diff coverage is 94.11%.

Impacted file tree graph

@@               Coverage Diff                @@
##           revamp_admin    #1183      +/-   ##
================================================
- Coverage         97.15%   97.09%   -0.06%     
================================================
  Files               422      423       +1     
  Lines              7056     7089      +33     
================================================
+ Hits               6855     6883      +28     
- Misses              201      206       +5
Impacted Files Coverage Δ
...ecidim/admin/update_participatory_process_admin.rb 100% <100%> (ø)
decidim-admin/config/routes.rb 100% <100%> (ø) ⬆️
...min/participatory_process_user_roles_controller.rb 93.47% <88.88%> (-2.96%) ⬇️
...ands/decidim/admin/update_participatory_process.rb 85.71% <0%> (-14.29%) ⬇️

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 4568f19...b291204. Read the comment docs.

@lastpotion lastpotion changed the title Update/admin processes [New Admin] Update process admins views. Mar 27, 2017
@mention-bot
Copy link
Copy Markdown

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

@lastpotion lastpotion force-pushed the update/admin_processes branch from 1ffb538 to 1e65e1b Compare March 28, 2017 07:47
create:
error: There was an error adding an admin for this participatory process.
success: Admin created successfully for this participatory process.
error: There was an error adding an admin role for this participatory process.
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.

The fact that this is a role is something internal, users are creating participatory process admins, not roles.

@lastpotion lastpotion merged commit 2bd9fcf into revamp_admin Mar 28, 2017
@lastpotion lastpotion deleted the update/admin_processes branch March 28, 2017 09:09
beagleknight pushed a commit that referenced this pull request Mar 28, 2017
* Add new and edit forms

* clean update command

* Tests added

* Amend test

* Add dashboard content

* Spellcheck

* Amend managing roles strings

* Use user instead of admin
beagleknight pushed a commit that referenced this pull request Mar 29, 2017
* Add new and edit forms

* clean update command

* Tests added

* Amend test

* Add dashboard content

* Spellcheck

* Amend managing roles strings

* Use user instead of admin
beagleknight pushed a commit that referenced this pull request Mar 30, 2017
* Add admin styles

* Add basic layout

* Change scopes icon

* Add participatory processes index page

* Add participatory process layout

* Fix some specs

* Fix more broken tests

* Fix more specs

* Fix poltergeist

* New admin - Settings section (#1124)

* Add participatory process layout

* Add basic layout for settings

* Finish settings form

* Add scopes index page

* Finish basic scopes admin

* Add revamp_admin to travis

* Fix a few specs

* Fix broken tests

* Remove bad merge

* [New admin] - Pages section (#1130)

* Add index section

* Add new section

* Finish forms

* Fix specs

* [New admin] - Users (#1136)

* Add layout

* Add users index page

* Add invite user form

* Add user groups index and fix pagination for both users and user groups

* Fix table links

* Fix specs

* Fix admin invite user labels

* [New admin] - Newsletters (#1140)

* Add layout and index page

* Add new/edit forms

* Add newsletter preview

* Fix specs

* [New Admin] - Process groups (#1141)

* Add new views

* Fix broken specs

* [New Admin] - Style process features (#1143)

* Style process features

* Refactor feature tests for admin

* [New Admin] - Processes info / steps (#1138)

* updated new process view

* updated process steps index

* update process steps index table

* update processes layouts

* Processes steps views updated

* Fix indentation

* Remove unusued locales

* Update tests

* Add moderations missing menu

* missing td added

* Add icon link helper (#1155)

* Add icon link helper

* Update existing icon links

* Add optional data to icon link helper

* Add methods at icon links

* Refactor tests for new icon helper

* Add docs and refactor helper call

* Refactor all icon links with new helper call

* [New Admin] Update category views. (#1148)

* Add category views

* Fix tests

* Add tests

* Add new icon_helper

* [New Admin] Attachment update. (#1158)

* Views added

* Finish tests

* add helper

* Style process features at admin (#1149)

* [New Admin] - Process groups (#1141)

* Add new views

* Fix broken specs

* Style meetings table

* Update branch with icon helper

* Style proposals

* Style results

* Style budgets

* Update missing styles for some forms

* Fix most of the tests

* Fix more tests

* Style moderation tables (#1182)

* Remove unused travis step

* [New Admin] Update process admins views. (#1183)

* Add new and edit forms

* clean update command

* Tests added

* Amend test

* Add dashboard content

* Spellcheck

* Amend managing roles strings

* Use user instead of admin

* Fix participatory process edit button

* Add some feedback

* Add styles for feature forms

* Add edit feature form

* Add more feedback

* [New Admin] Html and styles review. (#1200)

* Review all the forms

* Class coherence review

* CA: Swap funcionalitat with component

* Fix a few specs

* ES: Swap funcionalidad with componente

* Fix a few more specs

* Add missing helper

* Add preview images to organization settings form

* Icons updated on features table

* amend icon
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