Skip to content

fix:no history entry when changing RFC Editor note for doc#6021

Merged
rjsparks merged 2 commits intoietf-tools:mainfrom
PriyankaGitCom:fix-history-entry
Jul 23, 2023
Merged

fix:no history entry when changing RFC Editor note for doc#6021
rjsparks merged 2 commits intoietf-tools:mainfrom
PriyankaGitCom:fix-history-entry

Conversation

@PriyankaGitCom
Copy link
Copy Markdown
Contributor

No description provided.

Comment thread ietf/doc/views_ballot.py Outdated
e.by = login
e.type = "changed_rfc_editor_note_text"
e.desc = "RFC Editor Note was changed"
e.desc = "RFC Editor Note was changed to " + t
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think we need a newline before t starts. I suggest using
f"RFC Editor Note was changed to \n{t}" instead.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done the changes.

@codecov
Copy link
Copy Markdown

codecov Bot commented Jul 23, 2023

Codecov Report

Merging #6021 (7c45dac) into main (09f3477) will decrease coverage by 0.04%.
The diff coverage is 85.71%.

@@            Coverage Diff             @@
##             main    #6021      +/-   ##
==========================================
- Coverage   88.67%   88.64%   -0.04%     
==========================================
  Files         288      288              
  Lines       40001    40004       +3     
==========================================
- Hits        35471    35460      -11     
- Misses       4530     4544      +14     
Impacted Files Coverage Δ
ietf/doc/views_stats.py 74.80% <ø> (ø)
ietf/ietfauth/widgets.py 84.61% <ø> (ø)
ietf/nomcom/views.py 92.90% <33.33%> (-0.22%) ⬇️
ietf/doc/utils.py 87.15% <100.00%> (+0.01%) ⬆️
ietf/doc/views_ballot.py 92.86% <100.00%> (ø)
ietf/nomcom/templatetags/nomcom_tags.py 70.83% <100.00%> (+3.64%) ⬆️
ietf/submit/forms.py 79.02% <100.00%> (+0.03%) ⬆️

... and 7 files with indirect coverage changes

@rjsparks rjsparks merged commit 00c0389 into ietf-tools:main Jul 23, 2023
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Jul 26, 2023
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.

2 participants