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

fix: angular deliver preset #215

Merged
merged 1 commit into from
Jan 22, 2024
Merged

Conversation

jotanarciso
Copy link
Collaborator

@jotanarciso jotanarciso commented Jan 22, 2024

Problem

In the new version of Angular (which is initialized by default by the Azion CLI) it has SSR functionalities. As a result, the folder structure resulting from the build has a new subdivision: Browser and Server. We currently only support code on the Browser side (in deliver mode).

Resolution

This PR moves the contents of the '.edge/storage/browser' folder to the root of the storage (.edge/storage/) and displays an alert if the user is using SSR resources.

@jotanarciso jotanarciso changed the base branch from main to stage January 22, 2024 18:16
@jotanarciso jotanarciso changed the title fuxangular deliver preset fix: angular deliver preset Jan 22, 2024
lmalheiro
lmalheiro previously approved these changes Jan 22, 2024
Copy link
Contributor

@lmalheiro lmalheiro left a comment

Choose a reason for hiding this comment

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

LGTM. 🚀

Copy link
Contributor

@PatrickMenoti PatrickMenoti left a comment

Choose a reason for hiding this comment

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

🤝

@jotanarciso jotanarciso merged commit 5bb2d77 into stage Jan 22, 2024
@jotanarciso jotanarciso deleted the fix/angular-deliver-preset branch January 22, 2024 19:06
tiagokrebs pushed a commit that referenced this pull request Jan 22, 2024
[skip ci]
## [2.3.0-stage.3](v2.3.0-stage.2...v2.3.0-stage.3) (2024-01-22)

### Bug Fixes

* angular deliver preset ([#215](#215)) ([5bb2d77](5bb2d77))
* angular preset (deliver) ([f8c12b6](f8c12b6))
@tiagokrebs
Copy link
Contributor

🎉 This PR is included in version 2.3.0-stage.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

tiagokrebs pushed a commit that referenced this pull request Jan 22, 2024
[skip ci]
## [2.3.0](v2.2.0...v2.3.0) (2024-01-22)

### Features

* added building with external api to local runtime ([52bcd13](52bcd13))
* adding partial implementation of async_hooks to local dev ([3665359](3665359))
* resolve external APIs in the local build ([#211](#211)) ([b265ed6](b265ed6))

### Bug Fixes

* angular deliver preset ([#215](#215)) ([5bb2d77](5bb2d77))
* angular preset (deliver) ([f8c12b6](f8c12b6))
* remove versionid ([482fb35](482fb35))
* remove versionid ([#210](#210)) ([e78ea1e](e78ea1e))
@tiagokrebs
Copy link
Contributor

🎉 This PR is included in version 2.3.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

6 participants