Skip to content

[HUDI-1985] favicon plus misc site fixes#3388

Merged
vinothchandar merged 1 commit intoapache:asf-sitefrom
vingov:HUDI-1985_favicon_fix
Aug 3, 2021
Merged

[HUDI-1985] favicon plus misc site fixes#3388
vinothchandar merged 1 commit intoapache:asf-sitefrom
vingov:HUDI-1985_favicon_fix

Conversation

@vingov
Copy link

@vingov vingov commented Aug 3, 2021

Tips

What is the purpose of the pull request

  • Fixed favicon on the hudi.apache.org site.
  • Move the Our Vision Link to the top of the about footer menu.
  • Removed the last updated author and time for all the pages since it's showing my name for all the pages which are not correct plus removed the unnecessary CSS.
  • Fixed the Travis build to rebase before the merge.

Brief change log

  • Fixed a bunch of minor issues on the hudi asf site.

Verify this pull request

(Please pick either of the following options)

This change added tests and can be verified as follows:

Run these commands to test the site locally at http://localhost:3000/:
Note: ensure you have npm installed

./website/scripts/build-site.sh
cd website
npm run serve

Manually verified the website re-design changes by running the site locally.

Committer checklist

  • Has a corresponding JIRA in PR title & commit

  • Commit message is descriptive of the change

  • CI is green

  • Necessary doc changes done or have another open PR

  • For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

@vinothchandar vinothchandar merged commit f44880c into apache:asf-site Aug 3, 2021
@danny0405
Copy link
Contributor

image

This also need fix: => Spark Guide

@vinothchandar
Copy link
Member

Missed your comment by a minute or so :(

@vingov
Copy link
Author

vingov commented Aug 3, 2021

@danny0405 - I will review and fix all the titles and send the PR soon, I preserved most of the original titles but that doesn't make sense with this new navigation links.

@vingov vingov deleted the HUDI-1985_favicon_fix branch August 4, 2021 04:47
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