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

[DOC] Add new documentation for IP2Geo #4998

Merged
merged 3 commits into from
Sep 19, 2023
Merged

[DOC] Add new documentation for IP2Geo #4998

merged 3 commits into from
Sep 19, 2023

Conversation

vagimeli
Copy link
Contributor

@vagimeli vagimeli commented Sep 8, 2023

Description

Approved version; replaces the previous PR #4095 (which has merge main issues)

Issues Resolved

#3524

Checklist

  • By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
    For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@vagimeli vagimeli added 6 - Done but waiting to merge PR: The work is done and ready to merge v2.10.0 labels Sep 8, 2023
@vagimeli vagimeli self-assigned this Sep 8, 2023
@vagimeli vagimeli linked an issue Sep 8, 2023 that may be closed by this pull request
4 tasks
Signed-off-by: Melissa Vagi <[email protected]>
@vagimeli vagimeli removed the 6 - Done but waiting to merge PR: The work is done and ready to merge label Sep 19, 2023
@vagimeli
Copy link
Contributor Author

@hdhalter Will you provide approval so I can merge this PR? It is a new PR of the approved #4095, which addd over 400 files when it updated the branch. Those 400 files should not be part of the merge. This PR is to merge only the approved files for IP2Geo processor, which were already approved by doc and editorial in #4095 .

@vagimeli vagimeli merged commit 0fcc7d7 into main Sep 19, 2023
4 checks passed
@vagimeli vagimeli deleted the IP2Geo branch September 19, 2023 18:05
vagimeli added a commit that referenced this pull request Sep 19, 2023
* Approved through tech, doc, and editorial

Signed-off-by: Melissa Vagi <[email protected]>

* Publish documentation

Signed-off-by: Melissa Vagi <[email protected]>

---------

Signed-off-by: Melissa Vagi <[email protected]>
vagimeli added a commit that referenced this pull request Sep 19, 2023
harshavamsi pushed a commit to harshavamsi/documentation-website that referenced this pull request Oct 31, 2023
* Approved through tech, doc, and editorial

Signed-off-by: Melissa Vagi <[email protected]>

* Publish documentation

Signed-off-by: Melissa Vagi <[email protected]>

---------

Signed-off-by: Melissa Vagi <[email protected]>
vagimeli added a commit that referenced this pull request Dec 21, 2023
* Approved through tech, doc, and editorial

Signed-off-by: Melissa Vagi <[email protected]>

* Publish documentation

Signed-off-by: Melissa Vagi <[email protected]>

---------

Signed-off-by: Melissa Vagi <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[DOC]GeoIP processor
2 participants