Skip to content

Conversation

adambkaplan
Copy link
Member

@adambkaplan adambkaplan commented May 14, 2025

Changes

  • Add top-level site description, used to generate previews via meta tags.
  • Place favicons in the locations that the lotusdocs theme expects.

Fixes #165

/kind bug

Submitter Checklist

  • Includes tests if functionality changed/was added
  • Includes docs if changes are user-facing
  • Set a kind label on this PR
  • Release notes block has been filled in, or marked NONE

See the contributor guide
for details on coding conventions, github and prow interactions, and the code review process.

Release Notes

fix: Correct rendering of site's favicon and description.

- Add top-level site description, used to generate previews via meta
  tags.
- Place favicons in the locations that the lotusdocs theme expects.

Signed-off-by: Adam Kaplan <[email protected]>
@openshift-ci openshift-ci bot added the kind/bug Categorizes issue or PR as related to a bug. label May 14, 2025
@pull-request-size pull-request-size bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label May 14, 2025
Copy link
Contributor

openshift-ci bot commented May 14, 2025

@rxinui: changing LGTM is restricted to collaborators

In response to this:

/lgtm

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@rxinui
Copy link
Contributor

rxinui commented May 14, 2025

@adambkaplan just a comment.

You still need to edit the partial layout like I did here https://github.com/shipwright-io/website/pull/166/files#diff-96dd75a968976edd5e03170268ed9085733f75c3fb24f992ae613c89e6de42dcR1-R62

This is what defines the description in the link preview

@adambkaplan
Copy link
Member Author

Lotusdoc has a feature gap where we can't set the description using Hugo parameters.

Filed colinwilson/lotusdocs#239 as a feature request.

Copy link
Member

@SaschaSchwarze0 SaschaSchwarze0 left a comment

Choose a reason for hiding this comment

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

/approve
/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label May 19, 2025
Copy link
Contributor

openshift-ci bot commented May 19, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: SaschaSchwarze0

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label May 19, 2025
@openshift-merge-bot openshift-merge-bot bot merged commit a1fd4c7 into shipwright-io:main May 19, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. kind/bug Categorizes issue or PR as related to a bug. lgtm Indicates that a PR is ready to be merged. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[BUG] Previews in Slack Show Generic Template Content
3 participants