Skip to content

Fix filter by scopes#7858

Merged
mrcasals merged 5 commits intodecidim:developfrom
mainio:fix/filter_by_children_scopes
Apr 20, 2021
Merged

Fix filter by scopes#7858
mrcasals merged 5 commits intodecidim:developfrom
mainio:fix/filter_by_children_scopes

Conversation

@lahdeero
Copy link
Copy Markdown
Contributor

🎩 What? Why?

When a component has a parent scope defined for it, do not show the parent scope or the "general scope" in the filters

Testing

  1. Create a parent scope
  2. Create subscopes under the parent scope
  3. Create component e.g. proposals and enable parent scope
  4. Go to component's index view

📋 Checklist

🚨 Please review the guidelines for contributing to this repository.

  • CONSIDER adding a unit test if your PR resolves an issue.
  • ✔️ DO check open PR's to avoid duplicates.
  • ✔️ DO keep pull requests small so they can be easily reviewed.
  • ✔️ DO build locally before pushing.
  • ✔️ DO make sure tests pass.
  • ✔️ DO make sure any new changes are documented in docs/.
  • ✔️ DO add and modify seeds if necessary.
  • ✔️ DO add CHANGELOG upgrade notes if required.
  • ✔️ DO add to GraphQL API if there are new public fields.
  • ✔️ DO add link to MetaDecidim if it's a new feature.
  • AVOID breaking the continuous integration build.
  • AVOID making significant changes to the overall architecture.

📷 Screenshots

image

♥️ Thank you!

@mrcasals mrcasals added module: core type: fix PRs that implement a fix for a bug labels Apr 19, 2021
@mrcasals mrcasals merged commit 7b08408 into decidim:develop Apr 20, 2021
@ahukkanen ahukkanen deleted the fix/filter_by_children_scopes branch April 20, 2021 12:03
orlera pushed a commit that referenced this pull request Apr 22, 2021
orlera pushed a commit that referenced this pull request Apr 22, 2021
entantoencuanto added a commit that referenced this pull request Apr 23, 2021
* develop:
  Upgrade decidim-bulletin_board to 0.20.0 (#7881)
  New Crowdin updates (#7870)
  Fix editor: shift+enter makes single br tag before link (#7877)
  Fix TypeError in newsletters (#7872)
  Give admin panel access to monitoring committee members (#7843)
  New Crowdin updates (#7869)
  New Crowdin updates (#7864)
  Onboarding workflow after voting (#7839)
  Fix filter by scopes (#7858)
  Redirect unauthenticated users to sign in page for unauthorized views (#7852)
  New Crowdin updates (#7861)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module: core type: fix PRs that implement a fix for a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants