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: correct links to early meeting proceedings. #7492

Merged
merged 1 commit into from
Jun 4, 2024

Conversation

rjsparks
Copy link
Member

No description provided.

Copy link

codecov bot commented May 31, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.92%. Comparing base (187c2c5) to head (25bc7e3).
Report is 225 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7492      +/-   ##
==========================================
- Coverage   88.98%   88.92%   -0.07%     
==========================================
  Files         291      295       +4     
  Lines       40717    41224     +507     
==========================================
+ Hits        36233    36658     +425     
- Misses       4484     4566      +82     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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.

Looks fine, though I wonder if there'd be any value in teaching the agenda view to deal with these? (I guess it still needs a special case to know to give back a PDF instead of defaulting to HTML so maybe not worth it)

@rjsparks rjsparks merged commit 2c75df3 into ietf-tools:main Jun 4, 2024
9 checks passed
@rjsparks rjsparks deleted the earlyproc branch June 6, 2024 19:32
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 13, 2024
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