Skip to content

Conversation

@takahashim
Copy link
Contributor

In a Decidim instance with multiple organizations using DecidimAwesome, there are cases where information about components not belonging to the current organization is displayed in AwesomeMap. This PR fixes that issue.

@codecov
Copy link

codecov bot commented Jan 25, 2024

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (af92eb1) 96.47% compared to head (db63fe9) 96.48%.

Files Patch % Lines
...decidim/decidim_awesome/content_blocks/map_cell.rb 80.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #263   +/-   ##
=======================================
  Coverage   96.47%   96.48%           
=======================================
  Files         123      123           
  Lines        3465     3467    +2     
=======================================
+ Hits         3343     3345    +2     
  Misses        122      122           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@microstudi microstudi left a comment

Choose a reason for hiding this comment

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

Amazing, thanks a lot!

@microstudi microstudi merged commit 31af0cb into decidim-ice:main Jan 30, 2024
@takahashim takahashim deleted the fix-organization-check branch January 30, 2024 10:20
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.

2 participants