Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

Dynamically Generate Version Dropdown #18473

Merged
merged 4 commits into from
Jun 3, 2020

Conversation

ys2843
Copy link
Contributor

@ys2843 ys2843 commented Jun 2, 2020

Description

To make several website version dropdowns more manageable and easier to update, this PR includes the change to dynamically generate all version dropdown using the versions constant in _config.yml. This matches the implementation in global search #18288 (comment).
Updated version dropdown in this PR:

  • Installation guide dropdown
  • General version dropdown (on top nav)

Existing dynamically generate version dropdown:

  • 2 global search version dropdown(desktop & mobile)

Checklist

Essentials

Please feel free to remove inapplicable items for your PR.

  • The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
  • Changes are complete (i.e. I finished coding on this PR)
  • All changes have test coverage:

Changes

  • Update the general version dropdown on top navigation bar.
  • Update the installation guide version dropdown

Comments

@ys2843 ys2843 requested review from aaronmarkham and szha as code owners June 2, 2020 21:19
@mxnet-bot
Copy link

Hey @ys2843 , Thanks for submitting the PR
All tests are already queued to run once. If tests fail, you can trigger one or more tests again with the following commands:

  • To trigger all jobs: @mxnet-bot run ci [all]
  • To trigger specific jobs: @mxnet-bot run ci [job1, job2]

CI supported jobs: [edge, centos-gpu, website, unix-cpu, clang, miscellaneous, windows-cpu, centos-cpu, sanity, unix-gpu, windows-gpu]


Note:
Only following 3 categories can trigger CI :PR Author, MXNet Committer, Jenkins Admin.
All CI tests must pass before the PR can be merged.

@ys2843
Copy link
Contributor Author

ys2843 commented Jun 2, 2020

@mxnet-label-bot update [website, pr-awaiting-review]

@lanking520 lanking520 added Website pr-awaiting-review PR is waiting for code review labels Jun 2, 2020
@marcoabreu marcoabreu merged commit 691cf95 into apache:master Jun 3, 2020
@ys2843 ys2843 deleted the dynamic-general-version-dropdown branch June 3, 2020 20:35
@ys2843 ys2843 mentioned this pull request Jun 5, 2020
1 task
yijunc pushed a commit to yijunc/incubator-mxnet that referenced this pull request Jun 9, 2020
* dynamically generate general version dropdown

* dynamic generate installation guide version dropdown

* fix type

* use var

Co-authored-by: Yang Shi <[email protected]>
AntiZpvoh pushed a commit to AntiZpvoh/incubator-mxnet that referenced this pull request Jul 6, 2020
* dynamically generate general version dropdown

* dynamic generate installation guide version dropdown

* fix type

* use var

Co-authored-by: Yang Shi <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
pr-awaiting-review PR is waiting for code review Website
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants