xxubai opened a new pull request, #4105:
URL: https://github.com/apache/amoro/pull/4105
<!--
Thanks for sending a pull request!
Here are some tips for you:
1. If this is your first time, please read our contributor guidelines:
https://amoro.apache.org/how-to-contribute/
2. If the PR is related to an issue in
https://github.com/apache/amoro/issues, add '[AMORO-XXXX]' in your PR title,
e.g., '[AMORO-XXXX] Your PR title ...'.
3. If the PR is unfinished, add '[WIP]' in your PR title, e.g.,
'[WIP][AMORO-XXXX] Your PR title ...'.
-->
## Why are the changes needed?
<!--
Please clarify why the changes are needed. For instance,
1. If you add a feature, you can talk about its use case.
2. If you fix a bug, you can clarify why it is a bug.
3. Use Fix/Resolve/Close #{ISSUE_NUMBER} to link this PR to its related
issue
-->
This PR addresses multiple compliance issues identified by [ASF Whimsy
project checker](https://whimsy.apache.org/pods/project/amoro) to help the
Amoro project meet ASF graduation requirements for the project website.
Close #3832.
The following issues are fixed:
### 1. ❌ Events check — FAILED
> Projects SHOULD include a link to any current CommunityOverCode event, or
to the `events.apache.org` site, as provided by VP, Conferences.
The ASF navigation bar on the Amoro website was missing an **Events** link.
Per ASF policy, project websites must include a link to
`https://events.apache.org/`.
### 2. ⚠️ Resources check — WARNING
> Found 1 external resources: `ERROR Refused to load the stylesheet
'https://fonts.googleapis.com/css?family=Lato:300,400,700,300italic,400italic,700italic'`
because it violates the Content Security Policy directive.
Per the [ASF Privacy Policy for
Committers](https://privacy.apache.org/faq/committers.html), **websites must
not link to externally hosted resources**. The Amoro website was loading the
Lato font from Google Fonts CDN (`fonts.googleapis.com`), which violates this
policy and triggers a CSP error.
### 3. ❌ Image check — FAILED (manual action required)
> Projects SHOULD add a copy of their logo to
`https://www.apache.org/logos/` to be included in ASF homepage.
The Amoro project logo has not been submitted to the ASF project logos
repository yet. This requires a **manual SVN commit** (see documentation below)
and is not part of this PR's code changes.
**Reference:** https://whimsy.apache.org/pods/project/amoro
## Brief change log
### Commit 1: Add Events link to ASF navigation bar
- **`site/amoro-docs/hugo.toml`** — Added `Events` menu item
(`https://events.apache.org/`, weight 703) under the ASF navigation section;
re-numbered subsequent items (Donate → 704, Sponsors → 705, Security → 706,
Privacy → 707).
- **`site/amoro-site/hugo.toml`** — Same change applied to the landing page
site configuration.
### Commit 2: Self-host Lato font to eliminate external resource dependency
- **`site/amoro-theme/layouts/partials/css.html`** — Replaced the external
Google Fonts link:
```
- <link
href="//fonts.googleapis.com/css?family=Lato:300,400,700,300italic,400italic,700italic"
...>
+ <link href="{{ .Site.BaseURL }}/css/lato-font.css" ...>
```
- **`site/amoro-theme/static/css/lato-font.css`** *(new)* — Created local
`@font-face` declarations for all 6 Lato font variants (300/400/700 ×
normal/italic) using self-hosted WOFF2 files.
- **`site/amoro-theme/static/fonts/lato/*.woff2`** *(new, 6 files)* — Added
self-hosted Lato font files (latin subset, WOFF2 format) downloaded from Google
Fonts, licensed under the [SIL Open Font License](https://scripts.sil.org/OFL).
### Manual Action: Upload project logo to ASF (not in this PR)
The project logo (`site/amoro-theme/static/img/amoro-logo.svg`) needs to be
committed to the ASF SVN repository at
`https://svn.apache.org/repos/asf/comdev/project-logos/originals/amoro.svg` by
a committer with ASF SVN access. See [ASF Logos - How to
submit](https://www.apache.org/logos/about.html).
## How was this patch tested?
- [x] Verified the `Events` link appears correctly in the ASF dropdown
navigation on both the docs site and the landing page site.
- [x] Verified the Lato font loads correctly from self-hosted WOFF2 files
instead of Google Fonts CDN.
- [x] Confirmed all 6 WOFF2 font files are valid (verified `wOF2` magic
bytes) and cover all required weights/styles: Light (300), Regular (400), Bold
(700), and their italic variants.
- [x] Verified no remaining references to external resources
(`fonts.googleapis.com`) exist in the site theme templates.
## Documentation
- Does this pull request introduce a new feature? **No**
- If yes, how is the feature documented? **Not applicable** — these are
website compliance fixes for ASF graduation requirements.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
