Skip to content

Redesign: fix assemblies admin module issues#11702

Merged
andreslucena merged 4 commits intodevelopfrom
fix/assemblies-admin
Oct 4, 2023
Merged

Redesign: fix assemblies admin module issues#11702
andreslucena merged 4 commits intodevelopfrom
fix/assemblies-admin

Conversation

@alecslupu
Copy link
Copy Markdown
Contributor

🎩 What? Why?

This PR fixes the assemblies admin module issues revealed by #11600

📌 Related Issues

Link your PR to an issue

Testing

  • Extra pagination param should not be present in decidim-assemblies/app/views/decidim/assemblies/admin/assemblies/index.html.erb
  • Remove card-divider, and add pt-4 to .card in decidim-assemblies/app/views/decidim/assemblies/admin/assemblies_settings/_form.html.erb
  • Missing standardized header in decidim-assemblies/app/views/decidim/assemblies/admin/assemblies_settings/edit.html.erb
  • Remove card-divider, and add pt-4 to .card in decidim-assemblies/app/views/decidim/assemblies/admin/assemblies_types/new.html.erb
  • Missing standardized header in decidim-assemblies/app/views/decidim/assemblies/admin/assemblies_types/edit.html.erb
  • Remove card-divider, and add pt-4 to .card in decidim-assemblies/app/views/decidim/assemblies/admin/assembly_copies/_form.html.erb
  • Missing standardized header in decidim-assemblies/app/views/decidim/assemblies/admin/assembly_copies/new.html.erb
  • Title param is still passed to form and is not used in decidim-assemblies/app/views/decidim/assemblies/admin/assembly_copies/new.html.erb
  • Missing standardized header in decidim-assemblies/app/views/decidim/assemblies/admin/assembly_imports/new.html.erb
  • Remove card-divider, and add pt-4 to .card decidim-assemblies/app/views/decidim/assemblies/admin/assembly_imports/new.html.erb
  • Title param is still passed to form and is not used decidim-assemblies/app/views/decidim/assemblies/admin/assembly_imports/new.html.erb
  • Manage menu not consistent in decidim-assemblies/app/views/layouts/decidim/admin/assemblies_settings.html.erb
  • Manage menu not consistent in decidim-assemblies/app/views/layouts/decidim/admin/assemblies_types.html.erb
  • Old header present in decidim-assemblies/app/views/decidim/assemblies/admin/assembly_members/_form.html.erb
  • Title param is still passed to form and is not used in decidim-assemblies/app/views/decidim/assemblies/admin/assembly_members/edit.html.erb
  • Extra pagination param should not be present in decidim-assemblies/app/views/decidim/assemblies/admin/assembly_members/index.html.erb
  • Title param is still passed to form and is not used in decidim-assemblies/app/views/decidim/assemblies/admin/assembly_members/new.html.erb
  • Old header present should not be present in decidim-assemblies/app/views/decidim/assemblies/admin/assembly_user_roles/_form.html.erb
  • Title param is still passed to form and is not used in decidim-assemblies/app/views/decidim/assemblies/admin/assembly_user_roles/edit.html.erb
  • Title param is still passed to form and is not used in decidim-assemblies/app/views/decidim/assemblies/admin/assembly_user_roles/new.html.erb
  • Extra pagination param in decidim-assemblies/app/views/decidim/assemblies/admin/assembly_user_roles/index.html.erb
  • new Member Button in sidebar ( This is handled by Redesign: fix assembly members sidebar menu #11699) in decidim-assemblies/app/views/layouts/decidim/admin/assembly_members.html.erb

📷 Screenshots

Please add screenshots of the changes you are proposing
Description

♥️ Thank you!

@alecslupu alecslupu added this to the 0.28.0 milestone Oct 2, 2023
@alecslupu alecslupu marked this pull request as ready for review October 2, 2023 18:23
@alecslupu alecslupu requested a review from a team October 2, 2023 18:23
@andreslucena andreslucena changed the title Redesign: Fix assemblies admin module issues Redesign: fix assemblies admin module issues Oct 3, 2023
Copy link
Copy Markdown
Member

@andreslucena andreslucena left a comment

Choose a reason for hiding this comment

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

I found some details to change, can you give them a look?

Comment thread decidim-assemblies/app/views/layouts/decidim/admin/_manage_assemblies.html.erb Outdated
alecslupu and others added 2 commits October 3, 2023 14:38
Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>
@alecslupu alecslupu requested a review from andreslucena October 3, 2023 11:41
Copy link
Copy Markdown
Member

@andreslucena andreslucena left a comment

Choose a reason for hiding this comment

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

A bit more cleanup and this is ready to be merged

Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>
@alecslupu alecslupu requested a review from andreslucena October 3, 2023 13:40
Copy link
Copy Markdown
Member

@andreslucena andreslucena left a comment

Choose a reason for hiding this comment

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

💯

@andreslucena andreslucena merged commit bcfb78f into develop Oct 4, 2023
@andreslucena andreslucena deleted the fix/assemblies-admin branch October 4, 2023 08:06
@andreslucena andreslucena added type: fix PRs that implement a fix for a bug project: redesign Barcelona City Council contract no-backport Pull Requests that should not be backported labels Oct 4, 2023
entantoencuanto added a commit that referenced this pull request Oct 6, 2023
* feature/renaming-redesign:
  Use default current participatory space scope as root on scopes_select_tag called from bulk actions
  restore the imports thing (compilation fails for initiatives)
  change testing color: dequelabs/axe-core#3513 (comment)
  Fix admin redesign module (#11648)
  Apply flash styles to Announcements (#11708)
  Refactor oneliners of redesigned_a11y.js (#11713)
  Redesign: fix votings admin module issues (#11704)
  Add meta robots noindex to search and profile (#10120)
  Refactor dropdown scroll to menu (#11710)
  Remove unused partial
  Remove REDESIGN_PENDING obsolete comand
  Redesign: fix assembly members sidebar menu (#11699)
  Redesign: fix conferences admin module issues (#11703)
  Redesign: fix assemblies admin module issues (#11702)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module: assemblies no-backport Pull Requests that should not be backported project: redesign Barcelona City Council contract type: fix PRs that implement a fix for a bug

Projects

No open projects
Archived in project

Development

Successfully merging this pull request may close these issues.

Assemblies Admin module Issues

2 participants