Skip to content

PAF approval notifications and permissions - #2935

Merged
sandeepsajan0 merged 10 commits into
feature-project-approval-formfrom
paf-approval-notifications-permissions
Aug 9, 2022
Merged

PAF approval notifications and permissions #2935
sandeepsajan0 merged 10 commits into
feature-project-approval-formfrom
paf-approval-notifications-permissions

Conversation

@sandeepsajan0

@sandeepsajan0sandeepsajan0 commented Jul 22, 2022

Copy link
Copy Markdown
Member

Fixes#2916
Fixes#2913
Fixes#2920
Add description here.

@codecov

codecovBot commented Jul 24, 2022

Copy link
Copy Markdown

Codecov Report

Merging #2935 (118d784) into feature-project-approval-form (88ec974) will increase coverage by 0.13%.
The diff coverage is 97.72%.

@@ Coverage Diff @@## feature-project-approval-form #2935 +/- ##
=================================================================
+ Coverage 81.80% 81.93% +0.13% 
=================================================================
Files 683 683 Lines 24898 24992 +94 =================================================================
+ Hits 20367 20477 +110 + Misses 4531 4515 -16 
Impacted FilesCoverage Δ
hypha/apply/projects/admin.py75.00% <ø> (ø)
hypha/apply/projects/forms/__init__.py100.00% <ø> (ø)
hypha/apply/projects/views/__init__.py100.00% <ø> (ø)
...ypha/apply/projects/templatetags/approval_tools.py71.42% <50.00%> (+1.05%)⬆️
hypha/apply/projects/models/project.py90.97% <80.00%> (+1.48%)⬆️
hypha/apply/projects/views/project.py87.87% <96.77%> (+2.69%)⬆️
hypha/apply/funds/admin.py85.34% <100.00%> (+0.12%)⬆️
hypha/apply/projects/forms/project.py98.37% <100.00%> (-0.14%)⬇️
hypha/apply/projects/tests/factories.py99.15% <100.00%> (+0.02%)⬆️
hypha/apply/projects/tests/test_forms.py100.00% <100.00%> (ø)
... and 6 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@sandeepsajan0
sandeepsajan0 marked this pull request as ready for review July 26, 2022 08:14
Sign up for freeto 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.

1 participant

@sandeepsajan0