Skip to content

Information message when there isn't any Proposal#6063

Merged
microstudi merged 19 commits intodevelopfrom
feature/add_information_message_when_there_are_not_proposals
May 27, 2020
Merged

Information message when there isn't any Proposal#6063
microstudi merged 19 commits intodevelopfrom
feature/add_information_message_when_there_are_not_proposals

Conversation

@gemmatm
Copy link
Copy Markdown
Contributor

@gemmatm gemmatm commented May 4, 2020

🎩 What? Why?

This feature adds an information message when there isn't any Proposal or when the filtering doesn't find any Proposal with the filtered criteria.

📌 Related Issues

📋 Subtasks

  • Add CHANGELOG entry
  • Add tests

@gemmatm gemmatm self-assigned this May 4, 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.

Hi @gemmatm good job!
I only have a suggestion regarding indentations.
Can you also please take a look at linter checks? and resolve CHANGELOG.mf conflicts?

@iarviiis
Copy link
Copy Markdown
Contributor

molt be, @gemmatm

microstudi
microstudi previously approved these changes May 26, 2020
@microstudi microstudi merged commit 6186877 into develop May 27, 2020
@microstudi microstudi deleted the feature/add_information_message_when_there_are_not_proposals branch May 27, 2020 17:39
ace pushed a commit to aspgems/decidim that referenced this pull request May 28, 2020
* feature/add_areas_to_initiatives: (30 commits)
  Adds areas to FO filters
  Fix lint issue
  Fixes rubocop issues
  Updates changelog
  Adds areas to initiatives
  Send notification when signature threshold reached (decidim#6098)
  Adds filter by initiative type to admin panel (decidim#6093)
  Require confirmation on exiting a survey mid-answering (decidim#6118)
  Information message when there isn't any Proposal (decidim#6063)
  Set email asset host dynamically (decidim#5888)
  Harmonizes the design of initiatives search in FO (decidim#6090)
  Include year in meetings card (decidim#6102)
  Add attachment enabled option to initiative types and initiatives (decidim#6036)
  Fix a flaky test in group profile conversations (decidim#6123)
  Add attachments to Initiatives (decidim#5844)
  Add initiatives export (decidim#6070)
  Improvements to conversations with more than one participant (decidim#6094)
  Elections module and election administration (decidim#6065)
  Separate forms in steps (decidim#6108)
  Add sorting by publishing date to initiatives (decidim#6016)
  ...

# Conflicts:
#	decidim-initiatives/app/cells/decidim/initiatives/initiative_m_cell.rb
#	decidim-initiatives/app/commands/decidim/initiatives/admin/update_initiative.rb
#	decidim-initiatives/app/controllers/decidim/initiatives/initiatives_controller.rb
#	decidim-initiatives/app/forms/decidim/initiatives/admin/initiative_form.rb
#	decidim-initiatives/app/helpers/decidim/initiatives/application_helper.rb
#	decidim-initiatives/app/models/decidim/initiative.rb
#	decidim-initiatives/app/services/decidim/initiatives/initiative_search.rb
#	decidim-initiatives/app/views/decidim/initiatives/create_initiative/fill_data.html.erb
#	decidim-initiatives/app/views/decidim/initiatives/initiatives/_filters.html.erb
#	decidim-initiatives/app/views/decidim/initiatives/initiatives/_tags.html.erb
#	decidim-initiatives/config/locales/en.yml
#	decidim-initiatives/db/migrate/20200514085422_add_area_to_initiatives.rb
#	decidim-initiatives/db/migrate/20200514102631_add_area_enabled_option_to_initiatives.rb
#	decidim-initiatives/spec/forms/initiative_form_spec.rb
#	decidim-initiatives/spec/services/decidim/initiatives/initiative_search_spec.rb
#	decidim-initiatives/spec/shared/update_initiative_type_example.rb
#	decidim-initiatives/spec/system/admin/admin_manages_initiatives_spec.rb
#	decidim-initiatives/spec/system/admin/initiative_types_controller_spec.rb
#	decidim-initiatives/spec/system/filter_initiatives_spec.rb
ace pushed a commit to aspgems/decidim that referenced this pull request May 29, 2020
* develop:
  Send notification when signature threshold reached (decidim#6098)
  Adds filter by initiative type to admin panel (decidim#6093)
  Require confirmation on exiting a survey mid-answering (decidim#6118)
  Information message when there isn't any Proposal (decidim#6063)
  Set email asset host dynamically (decidim#5888)
  Harmonizes the design of initiatives search in FO (decidim#6090)
anaghavl pushed a commit to codegram/decidim that referenced this pull request Jun 3, 2020
* Add information message

* Add translations

* Fix translations

* Update information message on the correct view

* Add spec proposals index

* Update spec for proposals index

* Fix indentation

* Update Changelog

* Update decidim-proposals/spec/system/index_proposals_spec.rb

Co-authored-by: Oliver Valls <oliver.vh@coditramuntana.com>

* Fix linter checks

* Change information message

* Update index_proposals_spec.rb

* Update index_proposals_spec.rb

Co-authored-by: Gemma <gemmatm@coditramuntana.com>
Co-authored-by: Oliver Valls <oliver.vh@coditramuntana.com>
Co-authored-by: Ivan Vergés <ivan@platoniq.net>
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