Skip to content

Conversation

@yoshuawuyts
Copy link
Member

This adds two new conditional headers. Relies on #220. Ref #99. Thanks!

@yoshuawuyts yoshuawuyts force-pushed the if-modified-since branch 2 times, most recently from 190e358 to 1359054 Compare August 8, 2020 13:40
@yoshuawuyts
Copy link
Member Author

Realized Last-Modified is similar as well, so added that too!

@yoshuawuyts yoshuawuyts changed the title Add conditional::{IfModifiedSince, IfUnmodifiedSince} Add conditional::{IfModifiedSince, IfUnmodifiedSince, LastModified} Aug 8, 2020
@goto-bus-stop goto-bus-stop changed the base branch from master to main August 9, 2020 12:58
@Fishrock123 Fishrock123 marked this pull request as ready for review September 14, 2020 18:19
@yoshuawuyts yoshuawuyts dismissed Fishrock123’s stale review September 19, 2020 22:11

implemented the fix!

Co-authored-by: Jeremiah Senkpiel <[email protected]>
@yoshuawuyts yoshuawuyts merged commit f93c100 into main Sep 19, 2020
@delete-merged-branch delete-merged-branch bot deleted the if-modified-since branch September 19, 2020 22:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants