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

README for the IP check policy #923

Merged
merged 3 commits into from
Oct 8, 2018
Merged

README for the IP check policy #923

merged 3 commits into from
Oct 8, 2018

Conversation

davidor
Copy link
Contributor

@davidor davidor commented Oct 4, 2018

Adds a README with examples for the IP check policy. Also, places the policy in the correct subsection of the changelog and adds a missing link to JIRA.

@davidor davidor added this to the 3.4 milestone Oct 4, 2018
@davidor davidor requested a review from a team as a code owner October 4, 2018 15:12
Also, add missing references to issues.
@mikz mikz requested a review from a team October 4, 2018 16:17
@mikz
Copy link
Contributor

mikz commented Oct 4, 2018

@3scale/qe would be good to get your feedback about this. Hopefully, it helps you understand how it works.

@mayorova
Copy link
Contributor

mayorova commented Oct 4, 2018

Nice, thank you @davidor ! 👍

@supriya-bharadwaj
Copy link

Hi @mikz , I am unable to edit the files. Do you want me to simply add comments and then you want to incorporate my feedback? Do let me know. Thanks :)

@mikz
Copy link
Contributor

mikz commented Oct 4, 2018

@supriya-bharadwaj sorry for that, just have the @3scale/documentation team write permissions to the repo, so you should be free to edit the files in this PR. But anyway, PR comments are fine anyway! Any kind of feedback is much appreciated. 👍

@pestanko
Copy link

pestanko commented Oct 5, 2018

@mikz Looks reasonable, but we will know more when we start with automation.

Copy link

@vramosp vramosp left a comment

Choose a reason for hiding this comment

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

👍

@mikz mikz merged commit 2e800db into master Oct 8, 2018
@mikz mikz deleted the ip-check-policy-readme branch October 8, 2018 19:33
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.

6 participants