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

feat: Can we provide a better review assignment email subject. #3760 #4769

Closed
wants to merge 17 commits into from

Conversation

evyncke
Copy link
Contributor

@evyncke evyncke commented Nov 17, 2022

About #3760

1/ email subject now contains the reviewer name + deadline

2/ copies is also sent to the review requester and also to all secretaries

As discussed in #3760 with @cjbc @Abhayakara @kivinen @jgscudder

@evyncke evyncke changed the title Can we provide a better review assignment email subject. #3760 feat: Can we provide a better review assignment email subject. #3760 Nov 17, 2022
@evyncke
Copy link
Contributor Author

evyncke commented Nov 17, 2022

It looks like that I should also have updated the tests...

@rjsparks
Copy link
Member

It looks like that I should also have updated the tests...

Do you want me to do that, or will you have time soon?

@evyncke
Copy link
Contributor Author

evyncke commented Nov 18, 2022

Happy to keep learning on the tests aspects in the coming weeks.

ietf/review/utils.py Outdated Show resolved Hide resolved
@rjsparks
Copy link
Member

Converting to draft while the above changes are in progress. Please mark it ready for review once they're in place.

evyncke and others added 15 commits March 19, 2023 14:07
…s#5372)

Co-authored-by: depfu[bot] <23717796+depfu[bot]@users.noreply.github.com>
…5371)

Co-authored-by: depfu[bot] <23717796+depfu[bot]@users.noreply.github.com>
…er (ietf-tools#5370)

Co-authored-by: depfu[bot] <23717796+depfu[bot]@users.noreply.github.com>
…etf-tools#5369)

Co-authored-by: depfu[bot] <23717796+depfu[bot]@users.noreply.github.com>
@evyncke evyncke changed the base branch from main to feat/postgres March 25, 2023 06:12
@codecov
Copy link

codecov bot commented Mar 25, 2023

Codecov Report

Merging #4769 (eb9808c) into feat/postgres (88a78cf) will increase coverage by 0.01%.
The diff coverage is 50.00%.

@@                Coverage Diff                @@
##           feat/postgres    #4769      +/-   ##
=================================================
+ Coverage          88.80%   88.81%   +0.01%     
=================================================
  Files                285      285              
  Lines              39525    39525              
=================================================
+ Hits               35101    35105       +4     
+ Misses              4424     4420       -4     
Impacted Files Coverage Δ
ietf/review/utils.py 89.57% <ø> (ø)
ietf/utils/test_utils.py 82.11% <50.00%> (ø)

... and 5 files with indirect coverage changes

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

@evyncke
Copy link
Contributor Author

evyncke commented Mar 25, 2023

To clean up the PR (messed up by wrong merge), I have created a new one #5415

@evyncke evyncke closed this Mar 25, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 31, 2023
@evyncke evyncke deleted the fix-3760 branch November 2, 2023 14:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants