Skip to content

Conversation

@1502shivam-singh
Copy link
Member

@1502shivam-singh 1502shivam-singh commented Aug 21, 2021

Apache APISIX website redesign (Sub-PR: 1)

This large feature addition will be done in 3 chained PRs which for easier reviews and merging. Please merge all these 3 sub PRs together, in ascending order for no breaking changes. Dependency flow is like PR3 depends on PR2, depends on PR1.

This is the first PR supposed to be merged

Link to next PR to link
PR2 : #495

Changes:

Addition of asset files, namely images, SVGs, videos fonts etc. and config files

@netlify
Copy link

netlify bot commented Aug 21, 2021

✔️ Deploy Preview for apisix ready!

🔨 Explore the source changes: 85eeb4f

🔍 Inspect the deploy log: https://app.netlify.com/sites/apisix/deploys/61218a5d928a340007c3fd63

😎 Browse the preview: https://deploy-preview-493--apisix.netlify.app

@1502shivam-singh 1502shivam-singh changed the title feat: apisix-website redesign, added asset and config files feat: apisix-website redesign, added asset and config files (Sub-PR1) Aug 21, 2021
@1502shivam-singh 1502shivam-singh marked this pull request as ready for review August 22, 2021 00:12
@1502shivam-singh
Copy link
Member Author

cc @Yiyiyimu @liuxiran @juzhiyuan for review.

@juzhiyuan juzhiyuan requested a review from bzp2010 August 22, 2021 03:23
@1502shivam-singh
Copy link
Member Author

1502shivam-singh commented Aug 22, 2021

This PR actually doesn't have much to review, given that it just adds some static assets, like images, SVGs and fonts (still, if you want to, feel free to checkout files added).
Reviewers might want to checkout the next dependent PR #495 for reviewing actual code.

@juzhiyuan juzhiyuan requested a review from liuxiran August 22, 2021 23:27
@juzhiyuan juzhiyuan merged commit d98f3a7 into apache:master Aug 24, 2021
1502shivam-singh added a commit to 1502shivam-singh/apisix-website that referenced this pull request Sep 19, 2021
1502shivam-singh added a commit to 1502shivam-singh/apisix-website that referenced this pull request Sep 19, 2021
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