Skip to content

docs: tweak intro, add note w/ browser enforcement, FAQ #385

Merged
jonchurch merged 5 commits intoexpressjs:masterfrom
jonchurch:docs/add-misconceptions-and-important
Jan 16, 2026
Merged

docs: tweak intro, add note w/ browser enforcement, FAQ #385
jonchurch merged 5 commits intoexpressjs:masterfrom
jonchurch:docs/add-misconceptions-and-important

Conversation

@jonchurch
Copy link
Member

This PR is part of an attempt to reduce confusion in the docs #384

This adds a block at the top of the docs to try and call out the simplest version of what CORS is and what to expect from the package

Also adds a misconceptions section we can link to when needed

- Rewrite intro to clarify this middleware sets response headers
- Add [!IMPORTANT] callout explaining how CORS actually works
- Add Common Misconceptions section addressing common confusions
- Replace web.dev link with MDN reference in callout
@jonchurch jonchurch force-pushed the docs/add-misconceptions-and-important branch from eca5c8b to 5d39329 Compare January 12, 2026 23:26
@jonchurch jonchurch changed the title docs: add !important block w/ browser enforcement, FAQ docs: tweak intro, add note w/ browser enforcement, FAQ Jan 12, 2026
@jonchurch jonchurch added the docs label Jan 12, 2026
Copy link
Member

@UlisesGascon UlisesGascon left a comment

Choose a reason for hiding this comment

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

LGTM! Just a small suggestion to bold some parts (totally optional)

jonchurch and others added 3 commits January 14, 2026 11:03
Co-authored-by: Ulises Gascón <ulisesgascongonzalez@gmail.com>
Co-authored-by: Ulises Gascón <ulisesgascongonzalez@gmail.com>
Co-authored-by: Ulises Gascón <ulisesgascongonzalez@gmail.com>
@jonchurch jonchurch merged commit cf8947e into expressjs:master Jan 16, 2026
32 checks passed
@jonchurch jonchurch deleted the docs/add-misconceptions-and-important branch January 16, 2026 14:19
This was referenced Jan 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants