Skip to content

docs: Added Keyword API Gateway#8966

Closed
Ajay-singh1 wants to merge 70 commits into
apache:masterfrom
Ajay-singh1:master
Closed

docs: Added Keyword API Gateway#8966
Ajay-singh1 wants to merge 70 commits into
apache:masterfrom
Ajay-singh1:master

Conversation

@Ajay-singh1

@Ajay-singh1 Ajay-singh1 commented Feb 28, 2023

Copy link
Copy Markdown
Contributor

Description

Added Keyword API Gateway

Better SEO of the site

Fixes #7218

Checklist

  • I have explained the need for this PR and the problem it solves
  • I have explained the changes or the new features added to this PR
  • I have added tests corresponding to this change
  • I have updated the documentation to reflect this change
  • I have verified that this change is backward compatible (If not, please discuss on the APISIX mailing list first)

@Neilblaze

Neilblaze commented Mar 1, 2023

Copy link
Copy Markdown
Contributor

@Ajay-singh1 Thanks! But keeping the limelight on making the SEO better, apart from fixing typos and fixing translation issues, what made you add API Gateway to all the above pages? Do give a read where exactly API Gateway is being used and then update accordingly.

Once done, could you squash all commit messages into a single one?

cc: @navendu-pottekkat

@Ajay-singh1

Copy link
Copy Markdown
Contributor Author

Sure!

@Neilblaze Neilblaze left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@Ajay-singh1 Also, before you do that, switch to the master branch, update it to upstream, create a new branch from the master, make changes, commit, and create the pull request.

This was referenced Mar 1, 2023

@pottekkat pottekkat left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Adding to the keywords list might be good, but I don't see the need to forcefully change APISIX to APISIX API Gateway everywhere.

@Ajay-singh1

Copy link
Copy Markdown
Contributor Author

Adding to the keywords list might be good, but I don't see the need to forcefully change APISIX to APISIX API Gateway everywhere.

I have rectified it.

@Neilblaze

Copy link
Copy Markdown
Contributor

@Ajay-singh1 I'm repeating, follow this first and then squash all commits into a single one. Make sure to perform a Sign-Off.

If you can't then lemme know, I'll cc you in a different PR which I'll open once you confirm.

@Ajay-singh1

Copy link
Copy Markdown
Contributor Author

I don't know how to squash all the commits.I am afraid it can lead to merge conflicts.Can you redirect me to a different PR?

Thanks for your review.

@Neilblaze

Neilblaze commented Mar 2, 2023

Copy link
Copy Markdown
Contributor

Okay, I'll do it by eod (IST).

@Neilblaze

Neilblaze commented Mar 2, 2023

Copy link
Copy Markdown
Contributor

@Ajay-singh1 you can close this PR now. Make sure to close the duplicate ones too!

@Ajay-singh1

Copy link
Copy Markdown
Contributor Author

PR #8993 Has The Fix!Closing the PR.

@Ajay-singh1 Ajay-singh1 closed this Mar 2, 2023
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.

docs: add API Gateway keywords in English and Chinese docs

3 participants