Skip to content

Edit link in groups and show group in processes#6827

Merged
tramuntanal merged 4 commits intodevelopfrom
feature/process_groups_improvements-edit_link
Dec 9, 2020
Merged

Edit link in groups and show group in processes#6827
tramuntanal merged 4 commits intodevelopfrom
feature/process_groups_improvements-edit_link

Conversation

@entantoencuanto
Copy link
Copy Markdown
Contributor

🎩 What? Why?

  • Adds an edit link in processes groups public pages when user has permissions
  • Adds a link to the group in pages of processes which belongs to one

Note: This PR is built on top of feature/process_groups_improvements-participatory_processes_content_block, the branch of #6826 PR, but the changes introduced here are independent so they can be moved to a branch based on develop if necessary

📌 Related Issues

Link your PR to an issue

Testing

As admin visit the public page of a participatory process group
Visit the public page of a participatory process which belongs to a group

📋 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

Please add screenshots of the changes you're proposing
Description

♥️ Thank you!

@entantoencuanto entantoencuanto force-pushed the feature/process_groups_improvements-edit_link branch 3 times, most recently from e67adf2 to fb97d2c Compare November 10, 2020 16:07
border-right: $card-border;
border-right: 0;
border-bottom: $card-border;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Don't leave trailing whitespace

border-right: $card-border;
border-right: 0;
border-bottom: $card-border;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Unexpected whitespace at end of line (no-eol-whitespace)

@entantoencuanto entantoencuanto force-pushed the feature/process_groups_improvements-edit_link branch from fb97d2c to 1caae7a Compare November 11, 2020 16:37
@entantoencuanto entantoencuanto marked this pull request as ready for review November 16, 2020 12:58
@tramuntanal
Copy link
Copy Markdown
Contributor

There are conflicts to be resolved @entantoencuanto

@entantoencuanto entantoencuanto force-pushed the feature/process_groups_improvements-edit_link branch 2 times, most recently from f2bbb5f to b1ebe5f Compare November 30, 2020 19:12
@tramuntanal tramuntanal changed the title Feature/process groups improvements edit link in groups and show group in processes Edit link in groups and show group in processes Dec 3, 2020
@tramuntanal tramuntanal self-assigned this Dec 3, 2020
Copy link
Copy Markdown
Contributor

@tramuntanal tramuntanal left a comment

Choose a reason for hiding this comment

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

I started to review but there are so many repeated changes that's better to wait for #6826 to be merged.

@tramuntanal
Copy link
Copy Markdown
Contributor

tramuntanal commented Dec 4, 2020

Can you rebase please with develop ? @entantoencuanto

* Add test to check groups editable for admins
* Add admin edit link to group landing page
@entantoencuanto entantoencuanto force-pushed the feature/process_groups_improvements-edit_link branch from a401ade to 1de2817 Compare December 4, 2020 20:30
@entantoencuanto
Copy link
Copy Markdown
Contributor Author

Rebased

@tramuntanal tramuntanal merged commit 8def288 into develop Dec 9, 2020
@tramuntanal tramuntanal deleted the feature/process_groups_improvements-edit_link branch December 9, 2020 10:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Edit link as an Administrator in Process Groups

4 participants