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

fix: use Internet-Draft more consistently across the UI #5104

Merged
merged 13 commits into from
Feb 11, 2023

Conversation

larseggert
Copy link
Collaborator

@larseggert larseggert commented Feb 8, 2023

Fixes #4818

Copy link
Member

@rjsparks rjsparks left a comment

Choose a reason for hiding this comment

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

There's enough moving here that this deserves multiple reviews. I'm requesting one from Jennifer as well.

ietf/submit/utils.py Outdated Show resolved Hide resolved
ietf/sync/discrepancies.py Show resolved Hide resolved
ietf/templates/idindex/all_id2.txt Show resolved Hide resolved
ietf/utils/draft.py Outdated Show resolved Hide resolved
ietf/person/models.py Show resolved Hide resolved
ietf/review/models.py Show resolved Hide resolved
Copy link
Member

@jennifer-richards jennifer-richards left a comment

Choose a reason for hiding this comment

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

Did not find anything else alarming, but a couple comments inline.

ietf/templates/meeting/add_session_drafts.html Outdated Show resolved Hide resolved
ietf/name/fixtures/names.json Show resolved Hide resolved
@larseggert
Copy link
Collaborator Author

I think I need some help with the migrations - see CI errors.

@rjsparks
Copy link
Member

Not sure how you built these migrations - what you have at 0031 should have been in the review app and what you have at 0032 is a no-op?
In any case, here's the correction (which I'll push to your branch momentarily):

dev ➜ /workspace$ ietf/manage.py showmigrations | grep "\[ "
 [ ] 0031_id_term_2
 [ ] 0030_id_term_1
 [ ] 0032_id_term
dev ➜ /workspace$ ls ietf/person/migrations/003*py
0030_id_term_1.py  0031_id_term_2.py  0032_id_term.py    
dev ➜ /workspace$ rm ietf/person/migrations/003*py
dev ➜ /workspace$ ietf/manage.py makemigrations --name id_term
Migrations for 'person':
  ietf/person/migrations/0030_id_term.py
    - Alter field origin on email
    - Alter field origin on historicalemail
    - Alter field name_from_draft on historicalperson
    - Alter field name_from_draft on person
Migrations for 'review':
  ietf/review/migrations/0031_id_term.py
    - Alter field filter_re on historicalreviewersettings
    - Alter field filter_re on reviewersettings
dev ➜ /workspace$ ietf/manage.py migrate
Operations to perform:
  Apply all migrations: admin, auth, community, contenttypes, dbtemplate, django_celery_beat, doc, group, ipr, liaisons, mailinglists, mailtrigger, meeting, message, name, nomcom, oidc_provider, person, proceedings, review, sessions, sites, stats, submit, tastypie, utils
Running migrations:
  Applying person.0030_id_term... OK
  Applying review.0031_id_term... OK

It is nothing but coincidence that these are numbered 0030 and 0031 - it just turns out that's where the counters were in the respective apps.

@codecov
Copy link

codecov bot commented Feb 10, 2023

Codecov Report

Merging #5104 (c2de2a9) into main (e280b25) will increase coverage by 0.01%.
The diff coverage is 68.25%.

@@            Coverage Diff             @@
##             main    #5104      +/-   ##
==========================================
+ Coverage   88.50%   88.51%   +0.01%     
==========================================
  Files         295      295              
  Lines       40015    40015              
==========================================
+ Hits        35415    35420       +5     
+ Misses       4600     4595       -5     
Impacted Files Coverage Δ
ietf/doc/expire.py 95.31% <ø> (ø)
ietf/doc/views_ballot.py 92.86% <ø> (ø)
ietf/idindex/index.py 95.05% <ø> (ø)
ietf/ipr/utils.py 53.96% <0.00%> (ø)
ietf/review/models.py 91.91% <ø> (ø)
ietf/stats/views.py 93.10% <ø> (ø)
ietf/utils/draft.py 71.75% <0.00%> (ø)
ietf/doc/views_draft.py 91.94% <25.00%> (ø)
ietf/submit/forms.py 82.74% <37.50%> (ø)
ietf/submit/views.py 81.45% <50.00%> (ø)
... and 19 more

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

@rjsparks rjsparks merged commit f8b48f4 into ietf-tools:main Feb 11, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 17, 2023
@larseggert larseggert deleted the fix-4818 branch July 24, 2023 17:43
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.

Consistent use of terminology such as Internet-Draft and I-D
4 participants