Skip to content

docs: fix outdated astro:assets imports/snippets - #14541

Closed
sharadvc wants to merge 1 commit into
withastro:mainfrom
sharadvc:cursor/fix-astro-assets-docs-2f8d
Closed

sharadvc wants to merge 1 commit into
withastro:mainfrom
sharadvc:cursor/fix-astro-assets-docs-2f8d

Conversation

@sharadvc

Copy link
Copy Markdown

Summary

Update the astro:assets reference for current exports:

  • Drop removed astro/assets/utils helpers from imports/docs
  • Document propsToFilename / hashTransform under astro/assets
  • Fix <Image /> background snippet frontmatter
  • Document background on ImageTransform

English page only (keeps the PR small). Upgrade-guide emitImageMetadata wording left for a follow-up.

Fixes #14281

Update astro-assets reference to match current public exports: remove
utils helpers that are no longer exported, document propsToFilename and
hashTransform under astro/assets, fix the background example imports,
and document ImageTransform.background.

Fixes withastro#14281
@astrobot-houston

Copy link
Copy Markdown
Contributor

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/reference/modules/astro-assets.mdx Source changed, localizations will be marked as outdated.
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.

@ArmandPhilippot

Copy link
Copy Markdown
Member

As I said in #14529 this is an unwanted PR: the issue is labeled with should this be documented?. This is a big clue that there is nothing actionable yet. And if you had read the issue, the first part is already fixed.

So, I'm going to close this as well as your other PRs because you haven't listen me the first time:

Please refrain to submit automated PRs here and do not ignore maintainers comments.

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.

Incorrect snippet and outdated imports in astro:assets docs

3 participants