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

Update rate-limits.md. #495

Merged
merged 3 commits into from
Apr 30, 2019
Merged

Update rate-limits.md. #495

merged 3 commits into from
Apr 30, 2019

Conversation

jsha
Copy link
Contributor

@jsha jsha commented Apr 12, 2019

Clarify that renewals and duplicates are the same.

Add a note about the ordering issue that use to exist for the renewal
exemption.

Remove the note about gradually increasing total certificates for a
domain. Now that the limit is 50/week it's not as relevant.

Clarify that renewals and duplicates are the same.

Add a note about the ordering issue that use to exist for the renewal
exemption.
@tdelmas
Copy link
Collaborator

tdelmas commented Apr 13, 2019

lastmod is not updated, is it intentional? It's (especially) helps to tracks changes for translations.

@jsha
Copy link
Contributor Author

jsha commented Apr 14, 2019

Oops, that was not intentional! Thanks for catching. I'll fix that in my next push.

@jsha jsha requested a review from cpu April 30, 2019 17:37
Copy link
Contributor

@cpu cpu left a comment

Choose a reason for hiding this comment

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

I think this is much clearer. Nice work @jsha.

I flagged one punctuation nit to address.

Copy link
Contributor

@cpu cpu left a comment

Choose a reason for hiding this comment

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

👍

@Greek64
Copy link

Greek64 commented Apr 30, 2019

closes #439

@bdaehlie bdaehlie merged commit 6ae7441 into master Apr 30, 2019
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.

5 participants