Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(alpinejs): allow customizing the Alpine instance #9751

Merged
merged 8 commits into from
Jan 24, 2024

Conversation

florian-lefebvre
Copy link
Member

@florian-lefebvre florian-lefebvre commented Jan 21, 2024

Changes

  • Allows extending Alpine using the new entrypoint configuration. Most of the logic is taken from the vue integration

Testing

Tested manually, and I also added e2e tests.

Docs

withastro/docs#6534

/cc @withastro/maintainers-docs for feedback!

Copy link

changeset-bot bot commented Jan 21, 2024

🦋 Changeset detected

Latest commit: a8af9b9

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions github-actions bot added the pkg: integration Related to any renderer integration (scope) label Jan 21, 2024
@florian-lefebvre florian-lefebvre changed the title feat(alpinejs): allows customzing the Alpine instance feat(alpinejs): allow customizing the Alpine instance Jan 21, 2024
@florian-lefebvre florian-lefebvre marked this pull request as ready for review January 22, 2024 19:01
Copy link
Contributor

@matthewp matthewp left a comment

Choose a reason for hiding this comment

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

lgtm! I like that this follows the similar pattern we have with Vue.

@florian-lefebvre
Copy link
Member Author

The only thing I'm not sure about is the option name, but it's really easy to update

Copy link
Member

@MoustaphaDev MoustaphaDev left a comment

Choose a reason for hiding this comment

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

lgtm!

@ematipico ematipico merged commit 1153331 into withastro:main Jan 24, 2024
13 checks passed
@ematipico
Copy link
Member

Ah shoot, I didn't notice that we requested a review from docs 😱 sorry!

@astrobot-houston astrobot-houston mentioned this pull request Jan 24, 2024
@florian-lefebvre florian-lefebvre deleted the feat/alpine-entrypoint branch January 24, 2024 14:49
ematipico pushed a commit that referenced this pull request Jan 25, 2024
* feat(alpinejs): allows customzing the Alpine instance

* chore: add e2e tests

* fix: rename script

* Update index.ts

* fix: lockfile
ematipico added a commit that referenced this pull request Jan 31, 2024
* i18n(domains): validation and updated logic (#9099)

* feat(i18n): domain with lookup table (#9112)

* chore: add changelog, fix types and enable experimental support in node/vercel

* rebase and update lock file

* chore: fix failing test

* Apply suggestions from code review

Co-authored-by: Sarah Rainsberger <[email protected]>
Co-authored-by:  Matthew Phillips <[email protected]>

* Update .changeset/tidy-carrots-jump.md

Co-authored-by: Sarah Rainsberger <[email protected]>

* wip

* chore: rebase, conflicts and tests

* update lock file

* chore: correct configuration

* chore: correct configuration

* fix: regressions

* chore: fix conflicts and add more tests

* chore: add more validation

* chore: more tests and add more restrictions

* fix changeset

* change and revert adapters

* add another restriction

* lock file update

* Update packages/astro/src/@types/astro.ts

Co-authored-by: Sarah Rainsberger <[email protected]>

* Update packages/astro/src/@types/astro.ts

Co-authored-by: Bjorn Lu <[email protected]>

* wat

* fix syntax error

* fix config example

* Fix for #9673 (#9680)

* Fix for #9673

* 🦋 add changeset file

* Update breezy-plants-smoke.md

Co-authored-by: Florian Lefebvre <[email protected]>

* ⚡️ simplified normalizeConfigPath

---------

Co-authored-by: Florian Lefebvre <[email protected]>

* Fix env var replacement for export const prerender (#9807)

* feat(alpinejs): allow customizing the Alpine instance (#9751)

* feat(alpinejs): allows customzing the Alpine instance

* chore: add e2e tests

* fix: rename script

* Update index.ts

* fix: lockfile

* [ci] format

* chore: use correct lock file

* chore: rebase

* fix regressions in tests

* fix regressions in tests

* fix build

* add description

* fix missing types

* chore: fix tests, again :D

* eslint

* Update packages/astro/src/@types/astro.ts

Co-authored-by: Nate Moore <[email protected]>

* chore: address feedback

* chore: fix regressions

* chore: refactor naming

* Update packages/astro/src/core/app/index.ts

Co-authored-by: Bjorn Lu <[email protected]>

* chore: address feedback

* update lock file

* chore: infer routing from options, not strategy

* merge from main

* merge from main

* Experimental support in vercel adapter

* Update packages/astro/src/@types/astro.ts

Co-authored-by: Sarah Rainsberger <[email protected]>

* Update packages/astro/src/@types/astro.ts

Co-authored-by: Sarah Rainsberger <[email protected]>

* Update .changeset/tidy-carrots-jump.md

Co-authored-by: Sarah Rainsberger <[email protected]>

* better changesets

* Updates both experimental.i18nDomains and i18ndomains for experimental strategy

* fix link syntax

* consistent tabs/spaces

* Update packages/astro/src/@types/astro.ts

Co-authored-by: Sarah Rainsberger <[email protected]>

* apply suggestion

---------

Co-authored-by: Sarah Rainsberger <[email protected]>
Co-authored-by: Matthew Phillips <[email protected]>
Co-authored-by: Bjorn Lu <[email protected]>
Co-authored-by: Lou Cyx <[email protected]>
Co-authored-by: Florian Lefebvre <[email protected]>
Co-authored-by: Florian Lefebvre <[email protected]>
Co-authored-by: Nate Moore <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pkg: integration Related to any renderer integration (scope)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants