Skip to content

Add Hostinger deployment guide#13835

Open
ArnasDon wants to merge 7 commits intowithastro:mainfrom
ArnasDon:deploy/hostinger
Open

Add Hostinger deployment guide#13835
ArnasDon wants to merge 7 commits intowithastro:mainfrom
ArnasDon:deploy/hostinger

Conversation

@ArnasDon
Copy link
Copy Markdown

@ArnasDon ArnasDon commented May 5, 2026

Description (required)

Adds a deploy guide for Hostinger covering both static site uploads (via hPanel File Manager / FTP) and Node.js (server-rendered) deployments via hPanel.

This is a resubmission of #13649, which was closed because the build was failing. The previous attempt had:

  • The Markdown H1 placed before the YAML frontmatter, which caused content-collection schema validation to fail (no frontmatter detected → required fields missing).
  • A logo: hostinger value with no matching key in src/data/logos.ts, which is validated via `z.enum(logoKeys).parse(val)` and fails the build.
  • No `public/logos/hostinger.svg` asset.
  • `supports: ['ssr', 'static']` while the body only documented SSR.
  • `i18nReady: false` (other deploy guides use `true`).

This PR fixes all of the above:

  • Removes the duplicate H1 (the `title:` field already provides it) and uses correct frontmatter ordering, matching neighboring guides like `netlify.mdx` and `sevalla.mdx`.
  • Adds a `hostinger` entry to `src/data/logos.ts` in alphabetical order.
  • Adds `public/logos/hostinger.svg` (optimized with svgo, ~2 KB).
  • Fills in the static-site deployment section so the `supports` claim matches the content.
  • Sets `i18nReady: true`.
  • Verified locally with `pnpm build` (6268 pages built, no errors) and `pnpm prettier --check` on the changed files.

Related issues & labels (optional)

Adds a deploy guide for Hostinger covering both static and Node.js
(server-rendered) deployments via hPanel, plus the corresponding
hostinger logo and a `logos.ts` entry to satisfy the deploy-page
schema.

Resubmission of withastro#13649 with build/schema issues fixed.
@netlify
Copy link
Copy Markdown

netlify Bot commented May 5, 2026

Deploy Preview for astro-docs-2 ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit cbffd44
🔍 Latest deploy log https://app.netlify.com/projects/astro-docs-2/deploys/69fae20df8b781000896b245
😎 Deploy Preview https://deploy-preview-13835--astro-docs-2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@astrobot-houston
Copy link
Copy Markdown
Contributor

astrobot-houston commented May 5, 2026

Lunaria Status Overview

🌕 This pull request will trigger status changes.

Learn more

By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.

You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File Note
en/guides/deploy/hostinger.mdx Localization added, will be marked as complete.
Warnings reference
Icon Description
🔄️ The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

Copy link
Copy Markdown
Author

@ArnasDon ArnasDon left a comment

Choose a reason for hiding this comment

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

No failing tests from automatic checks. Based on the previously closed PR, fixed them and opened a new PR.

@ArmandPhilippot ArmandPhilippot added add new content Document something that is not in docs. May require testing, confirmation, or affect other pages. 3rd-party guide Community submitted CMS, deploy, backend etc. guide labels May 5, 2026
Copy link
Copy Markdown
Member

@ArmandPhilippot ArmandPhilippot left a comment

Choose a reason for hiding this comment

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

Thank you! Overall, this looks good to me. I made a few suggestions, but they were mostly nitpicks. My biggest concern is the logo: does Hostinger as a smaller version that can be used (see my comment on the logo file)?

Comment thread public/logos/hostinger.svg
Comment thread src/content/docs/en/guides/deploy/hostinger.mdx Outdated
Comment thread src/content/docs/en/guides/deploy/hostinger.mdx Outdated
Comment thread src/content/docs/en/guides/deploy/hostinger.mdx Outdated
Comment thread src/content/docs/en/guides/deploy/hostinger.mdx Outdated
Comment thread src/content/docs/en/guides/deploy/hostinger.mdx Outdated
ArnasDon and others added 6 commits May 6, 2026 09:36
Replaces the wide wordmark SVG with the official Hostinger "h" icon
mark, which matches the icon-style format used by neighboring deploy
guides (Heroku, Microsoft Azure, etc.) and renders at a more readable
size in the deploy index card grid.
Co-authored-by: Armand Philippot <git@armand.philippot.eu>
Co-authored-by: Armand Philippot <git@armand.philippot.eu>
Co-authored-by: Armand Philippot <git@armand.philippot.eu>
Co-authored-by: Armand Philippot <git@armand.philippot.eu>
Co-authored-by: Armand Philippot <git@armand.philippot.eu>
Copy link
Copy Markdown
Member

@ArmandPhilippot ArmandPhilippot left a comment

Choose a reason for hiding this comment

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

Thank you for the updates. The logo seems better integrated now. 😄 LGTM!

@ArnasDon
Copy link
Copy Markdown
Author

ArnasDon commented May 6, 2026

Great @ArmandPhilippot, thanks for reviewing the changes. Is another approval still required with writes access or you can approve this PR to be merged?

@ArmandPhilippot
Copy link
Copy Markdown
Member

A second approval is not required, but we generally do so for new content or significant changes. So yes, I'm waiting for someone else's review before merging this PR. Yan will probably do it this week, when he has the time!

@ArnasDon
Copy link
Copy Markdown
Author

ArnasDon commented May 6, 2026

Gotcha! Thanks for the information.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3rd-party guide Community submitted CMS, deploy, backend etc. guide add new content Document something that is not in docs. May require testing, confirmation, or affect other pages.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants