Skip to content

fix: Don't allow ADs to set docs to announcement-sent states#6445

Merged
rjsparks merged 1 commit intoietf-tools:mainfrom
pselkirk:fix-5644
Oct 10, 2023
Merged

fix: Don't allow ADs to set docs to announcement-sent states#6445
rjsparks merged 1 commit intoietf-tools:mainfrom
pselkirk:fix-5644

Conversation

@pselkirk
Copy link
Copy Markdown
Contributor

@pselkirk pselkirk commented Oct 7, 2023

Fixes #5644

@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 7, 2023

Codecov Report

Merging #6445 (8e7608f) into main (40dc24e) will increase coverage by 0.00%.
Report is 29 commits behind head on main.
The diff coverage is 84.84%.

@@           Coverage Diff           @@
##             main    #6445   +/-   ##
=======================================
  Coverage   88.68%   88.69%           
=======================================
  Files         290      290           
  Lines       40431    40446   +15     
=======================================
+ Hits        35857    35872   +15     
  Misses       4574     4574           
Files Coverage Δ
ietf/doc/views_conflict_review.py 96.80% <100.00%> (+0.05%) ⬆️
ietf/doc/views_draft.py 91.97% <100.00%> (+0.03%) ⬆️
ietf/doc/views_review.py 95.16% <100.00%> (ø)
ietf/doc/views_status_change.py 93.45% <100.00%> (+0.07%) ⬆️
ietf/nomcom/urls.py 100.00% <ø> (ø)
ietf/nomcom/views.py 93.00% <ø> (-0.02%) ⬇️
ietf/review/utils.py 92.13% <75.00%> (ø)
ietf/review/policies.py 97.47% <50.00%> (ø)

... and 3 files with indirect coverage changes

@rjsparks rjsparks merged commit 72fef7b into ietf-tools:main Oct 10, 2023
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Oct 14, 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.

Area Directors should not be able to set the "Announcement Sent" for a document

2 participants