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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

[bug] multiple-forms example: missing import #466

Closed
petecartwright opened this issue Apr 26, 2024 · 0 comments 路 Fixed by #467
Closed

[bug] multiple-forms example: missing import #466

petecartwright opened this issue Apr 26, 2024 · 0 comments 路 Fixed by #467

Comments

@petecartwright
Copy link
Contributor

馃憢 hello friends!

Looks like some imports were accidentally deleted in #276.

Specifically, this deleteInvitiation function is no longer imported from ~/data.server. The app still builds with npm run build, but will error when running. (see gif below)

deleteInvitation not defined

and the corresponding error in the terminal:
terminal error

To reproduce:

  1. open /multiple-forms locally
  2. run npm i, npm run build, npm run start
  3. open the local app in your browser
  4. click "Delete" on any invitation on the list
  5. see the error, feel sad 馃槩

I'll be opening a PR shortly to fix this.

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 a pull request may close this issue.

1 participant