Skip to content

Document integration hooks for middleware#5253

Merged
sarah11918 merged 3 commits into
mainfrom
int-middleware
Nov 9, 2023
Merged

Document integration hooks for middleware#5253
sarah11918 merged 3 commits into
mainfrom
int-middleware

Conversation

@matthewp
Copy link
Copy Markdown
Contributor

@matthewp matthewp commented Nov 1, 2023

Description (required)

This documents the addMiddleware hook for integrations, part of the integration hooks RFC: withastro/roadmap#735

Related issues & labels (optional)

For Astro version: 3.5. See astro PR #8869.

@netlify
Copy link
Copy Markdown

netlify Bot commented Nov 1, 2023

Deploy Preview for astro-docs-2 ready!

Name Link
🔨 Latest commit c1c184e
🔍 Latest deploy log https://app.netlify.com/sites/astro-docs-2/deploys/654d33184bfdcd00088e3032
😎 Deploy Preview https://deploy-preview-5253--astro-docs-2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@sarah11918 sarah11918 added add new content Document something that is not in docs. May require testing, confirmation, or affect other pages. merge-on-release Don't merge this before the feature is released! (MQ=approved but WAIT for feature release!) minor-release For the next minor release; in the milestone, "merge queue" = approved for merging on release day. labels Nov 3, 2023
Copy link
Copy Markdown
Member

@sarah11918 sarah11918 left a comment

Choose a reason for hiding this comment

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

My fave kind of PRs... approved!

Copy link
Copy Markdown
Member

@yanthomasdev yanthomasdev left a comment

Choose a reason for hiding this comment

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

LGTM!

},
};
```
#### `addMiddleware` option
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Just noting that there was a merge comment since the Dev Overlay PR merged in the mean time. Should all be fixed now!

@sarah11918 sarah11918 merged commit c53ff79 into main Nov 9, 2023
@sarah11918 sarah11918 deleted the int-middleware branch November 9, 2023 19:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

add new content Document something that is not in docs. May require testing, confirmation, or affect other pages. merge-on-release Don't merge this before the feature is released! (MQ=approved but WAIT for feature release!) minor-release For the next minor release; in the milestone, "merge queue" = approved for merging on release day.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants