Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added source field to Thank You Note #1529

Merged
merged 1 commit into from
Nov 17, 2023
Merged

Added source field to Thank You Note #1529

merged 1 commit into from
Nov 17, 2023

Conversation

edwoodward
Copy link
Contributor

No description provided.

@edwoodward edwoodward self-assigned this Nov 17, 2023
@edwoodward edwoodward changed the base branch from main to development November 17, 2023 15:05
Copy link

codecov bot commented Nov 17, 2023

Codecov Report

Merging #1529 (38be7ae) into development (7d75f3b) will increase coverage by 2.96%.
Report is 73 commits behind head on development.
The diff coverage is 85.86%.

@@               Coverage Diff               @@
##           development    #1529      +/-   ##
===============================================
+ Coverage        78.93%   81.89%   +2.96%     
===============================================
  Files              125      116       -9     
  Lines             6380     6552     +172     
===============================================
+ Hits              5036     5366     +330     
+ Misses            1344     1186     -158     

@edwoodward edwoodward merged commit f349c58 into development Nov 17, 2023
2 checks passed
@edwoodward edwoodward deleted the ty-source branch November 17, 2023 15:51
mwvolo added a commit that referenced this pull request Dec 20, 2023
* Wagtail 4.1.9 upgrade (#1522)

* Updated to wagtail 4.1.9 and replaced WagtailPageTests with WagtailPageTestCase

* Fixes migration conflict

* Added source field to Thank You Note (#1529)

* Update admin.py (#1532)

Co-authored-by: Staxly <[email protected]>

* updating book filter to exclude (#1530)

Co-authored-by: Staxly <[email protected]>
Co-authored-by: Michael Volo <[email protected]>

---------

Co-authored-by: Ed Woodward <[email protected]>
Co-authored-by: Colby <[email protected]>
Co-authored-by: Staxly <[email protected]>
mwvolo added a commit that referenced this pull request Jan 2, 2024
* Wagtail 4.1.9 upgrade (#1522)

* Updated to wagtail 4.1.9 and replaced WagtailPageTests with WagtailPageTestCase

* Fixes migration conflict

* Added source field to Thank You Note (#1529)

* Update admin.py (#1532)

Co-authored-by: Staxly <[email protected]>

* updating book filter to exclude (#1530)

Co-authored-by: Staxly <[email protected]>
Co-authored-by: Michael Volo <[email protected]>

* remove unused account id field from resource downloads

* do not check for book and contact on resource creation

* remove rejection test

* Require UUID, do not require resource name

* set default uuid to something random

---------

Co-authored-by: Ed Woodward <[email protected]>
Co-authored-by: Colby <[email protected]>
Co-authored-by: Staxly <[email protected]>
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