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

Expose assets bucket name to static-website and single-page-app #311

Conversation

MarcEspiard
Copy link
Contributor

Hi,

Similar to #241, exposing the assets bucket name for the static website and spa constructs.
My use case:
I want to trigger an event after objects are created in the bucket so a Lambda can add some metadata to each file uploaded in that folder (Cache Control headers etc...)

@mnapoli
Copy link
Member

mnapoli commented Mar 2, 2023

Thanks for opening this PR.

@fredericbarthelet given we merged #241 I think it makes sense to merge this one too (same variable), WDYT?

@fredericbarthelet
Copy link
Collaborator

@mnapoli agreed :)

@MarcEspiard, could you update your branch to latest version of master ? We recently shipped a fix that should allow tests with latest version of serverless to pass on your PR :)

@MarcEspiard
Copy link
Contributor Author

MarcEspiard commented Mar 2, 2023

@fredericbarthelet Thank you for that, just updated with latest changes from master.
Are you able to do a release after this is merged please? Would really help with the project I'm currently working on.

@fredericbarthelet fredericbarthelet merged commit 96c8b37 into getlift:master Mar 3, 2023
@fredericbarthelet
Copy link
Collaborator

@MarcEspiard published in 1.24.0

Thanks for your contribution :)

@MarcEspiard
Copy link
Contributor Author

@fredericbarthelet no worries, bonne journée :)

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.

None yet

3 participants