Skip to content

[FLINK-33356][docs] Fix the messed navigation bar on Flink’s official website#23627

Merged
wanglijie95 merged 1 commit intoapache:masterfrom
WencongLiu:dev_FLINK-33356
Oct 31, 2023
Merged

[FLINK-33356][docs] Fix the messed navigation bar on Flink’s official website#23627
wanglijie95 merged 1 commit intoapache:masterfrom
WencongLiu:dev_FLINK-33356

Conversation

@WencongLiu
Copy link
Copy Markdown
Contributor

@WencongLiu WencongLiu commented Oct 31, 2023 •
edited
Loading

What is the purpose of the change

Fix the messed navigation bar on Flink’s official website.

Brief change log

  • Fix the messed navigation bar on Flink’s official website.

Currently, the website returns to normal.

image

Does this pull request potentially affect one of the following parts:

  • Dependencies (does it add or upgrade a dependency): (no)
  • The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
  • The serializers: (no)
  • The runtime per-record code paths (performance sensitive): (no)
  • Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no)
  • The S3 file system connector: (no)

Documentation

  • Does this pull request introduce a new feature? (no)
  • If yes, how is the feature documented? (no)

@flinkbot
Copy link
Copy Markdown
Collaborator

flinkbot commented Oct 31, 2023 •
edited
Loading

CI report:

Bot commands The @flinkbot bot supports the following commands:
  • @flinkbot run azure re-run the last Azure build

@JunRuiLee
Copy link
Copy Markdown
Contributor

JunRuiLee commented Oct 31, 2023

@WencongLiu Could you please attach a screenshot to show what the modified page looks like?

@WencongLiu
Copy link
Copy Markdown
Contributor Author

WencongLiu commented Oct 31, 2023

@WencongLiu Could you please attach a screenshot to show what the modified page looks like?

Done. I've attached a screenshot. Thanks for reminding me! 🙏 @JunRuiLee

JunRuiLee
JunRuiLee approved these changes Oct 31, 2023
View reviewed changes
Copy link
Copy Markdown
Contributor

@JunRuiLee JunRuiLee left a comment

Choose a reason for hiding this comment

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

LGTM

wanglijie95
wanglijie95 approved these changes Oct 31, 2023
View reviewed changes
Copy link
Copy Markdown
Contributor

@wanglijie95 wanglijie95 left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @WencongLiu

@wanglijie95 wanglijie95 merged commit 780a673 into apache:master Oct 31, 2023
@WencongLiu WencongLiu deleted the dev_FLINK-33356 branch October 31, 2023 08:54
KarmaGYZ pushed a commit to KarmaGYZ/flink that referenced this pull request Nov 20, 2023
@WencongLiu @KarmaGYZ
[FLINK-33356][docs] Fix the messed navigation bar on Flink’s official…
2ec1cbf 
… website

This closes apache#23627
KarmaGYZ pushed a commit to KarmaGYZ/flink that referenced this pull request Nov 20, 2023
@WencongLiu @KarmaGYZ
[hotfix][docs] Fix the messed navigation bar on Flink’s official website
76cd1e2 
This closes apache#23627
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@wanglijie95 wanglijie95 wanglijie95 approved these changes

@JunRuiLee JunRuiLee JunRuiLee approved these changes

Assignees

No one assigned

Labels

component=ProjectWebsite

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@WencongLiu @flinkbot @JunRuiLee @wanglijie95