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

Nextjs #63

Open
wants to merge 80 commits into
base: main
Choose a base branch
from
Open

Nextjs #63

wants to merge 80 commits into from

Conversation

JeffBaumgardt
Copy link
Contributor

@JeffBaumgardt JeffBaumgardt commented Sep 13, 2023

Very destructive merge. This overwrites everything from the remix site.

Jeff Baumgardt and others added 30 commits July 3, 2023 10:49
* converting contact to sign-up

* color text light mode

---------

Co-authored-by: mfolsom930 <[email protected]>
* Adding a responsive but non-functional event sign up form to the homepage.

* Made some alignment changes to the form and made the featured event slightly larger.

* Moved event signup form into event page and adjusted the styles for that layout.

* Moved the signup form above the event write up.

* Added in the structure of the API request from the current site and prepped as much as I could without the API token. Also switched out axios for use of fetch, but unable to troubleshoot until I'm able to successfully hit the API.

* Removing the date of the event that was added to the title by mistake.

* Adding nextjs API route to handle tito request. This commit has the first call in place, returning the next event slug, the virtual id, and the in person id.

* Added functionality to register user, also changed tito api file name to tito-check-slug.

* Added ticket link to UI

* Added date check to form to only render the sign up form if the event is coming up.

* Update pnpm lock

* Removed unnecessary next.config line.

* Adding prettier file

* Remove async from checkSlug function since its already using promises.

* Split out one API error into 2 and displayed these errors to the user if need be. Also changed an error throw to a console.error, so that a developer can troubleshoot an issue if need be.

* Removed extra call to tito API by passing the event slug in the URL in the checkSlug call.

* Removed checkSlug function from the event signup component and moved that functionality into /event/[slug]/page.js, then passed the event data into the event sign up component as a prop.

* Made the fetch call in onSubmit async so that the loader on the register button would show when posting a registration.

---------

Co-authored-by: Jeff Baumgardt <[email protected]>
Add a hover on speakers and orgs.
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.

2 participants