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

Initial form #8

Merged
merged 7 commits into from
Oct 27, 2023
Merged

Initial form #8

merged 7 commits into from
Oct 27, 2023

Conversation

danielnaab
Copy link
Contributor

@danielnaab danielnaab commented Oct 27, 2023

This PR creates a new frontend test page at /form-sample, which submits to a new AWS Lambda function that is wired up to API Gateway. It doesn't do anything interesting, yet, but can serve as the base for our initial backend form logic.

NOTE: In the process of implementing this, I discovered what looks like a peer dependency bug in pnpm. Since the introduction of Astro, transitive dependencies have somehow broken the Terraform CDK deployment. Removing Astro, temporarily, allows the cdktf synthesis to complete. In theory, this should not happen, due to the isolation provided by separate projects for infra and spotlight. This will require further investigation to track down a workaround to, but does not block any current or upcoming work.

@danielnaab danielnaab merged commit 5c10f32 into main Oct 27, 2023
1 check failed
@danielnaab danielnaab deleted the initial-form branch October 27, 2023 20:10
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.

1 participant