Skip to content

frontend: fix bug exclude_fundingOrganization=true - #1255

Merged
BibianaC merged 1 commit into
livefrom
bc/exclude_fundingOrganization=true
Jul 31, 2026
Merged

frontend: fix bug exclude_fundingOrganization=true#1255
BibianaC merged 1 commit into
livefrom
bc/exclude_fundingOrganization=true

Conversation

@BibianaC

Copy link
Copy Markdown
Member

Relates to #1232

@coveralls

coveralls commented May 20, 2026

Copy link
Copy Markdown

Coverage Report for CI Build 27194749515

Coverage increased (+0.1%) to 72.485%

Details

  • Coverage increased (+0.1%) from the base build.
  • Patch coverage: 1 uncovered change across 1 file (4 of 5 lines covered, 80.0%).
  • No coverage regressions found.

Uncovered Changes

File Changed Covered %
grantnav/frontend/views.py 4 3 75.0%
Total (2 files) 5 4 80.0%

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 1988
Covered Lines: 1441
Line Coverage: 72.48%
Coverage Strength: 0.72 hits per line

💛 - Coveralls

@BibianaC
BibianaC requested a review from chrisarridge May 20, 2026 09:21
@BibianaC
BibianaC force-pushed the bc/exclude_fundingOrganization=true branch from ad9cda9 to be13775 Compare June 9, 2026 08:47
@BibianaC
BibianaC requested a review from jwilson232 July 30, 2026 06:57
@BibianaC

Copy link
Copy Markdown
Member Author

@chrisarridge @jwilson232 Marion has tested this PR and is happy with it. Could you please review the code?

@jwilson232 jwilson232 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

looks good!

@BibianaC
BibianaC merged commit 7a8b5a2 into live Jul 31, 2026
5 of 7 checks passed
@BibianaC
BibianaC deleted the bc/exclude_fundingOrganization=true branch July 31, 2026 06:34
@mariongalley

Copy link
Copy Markdown
Contributor

Tested in live with GrantVis and confirmed the fix worked

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