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

[feat] implements basic interest form #10

Merged
merged 4 commits into from
Oct 6, 2023

Conversation

varortz
Copy link
Collaborator

@varortz varortz commented Oct 3, 2023

🎋 Description

Creates simple interest form.

🌴 What's new in this PR

  • updated the schema for objects of type Interest (added type string for id to facilitate interest inserts)
  • Created insertInterest and getAllInterests queries
  • Implemented a basic form that inserts an interest object (with hardcoded data) to the interests table in Supabase

🌲 Screenshots

image

🌳 How to review

  • Open the app in local host and navigate to http://localhost:3000/interest
  • Add dummy data into the textarea input, and press the submit button to complete the action
  • Go to the interests table in Supabase and confirm that a new record was added (can look at whether the record count increases)

🌱 Next steps

🔗 Relevant Links

ℹ️ Online sources

🪴 Related PRs

CC: @pragyakallanagoudar

@linear
Copy link

linear bot commented Oct 3, 2023

IJP-6 Create simple interest form.

Route: /interest (src/app/interest/page.tsx)

  • Add the schema for the interests table to Supabase using the table editor.
  • Upload Jinkang's mocked data (in CSV format) to this table (Stack Overflow).
  • Create a very simple case interest form with a prompt, a text field, and a button to submit. No styling!
  • When the button is pressed, write the response to the interest table on Supabase (documentation).
    • Hardcode user_id and listingId for right now. Generate a UUID for the interestId.

❗ We will be using TypeScript, not JavaScript!

@varortz varortz changed the title basic interest form complete [feat] Implements basic interest form Oct 5, 2023
package.json Outdated Show resolved Hide resolved
Copy link
Collaborator

@pragyakallanagoudar pragyakallanagoudar left a comment

Choose a reason for hiding this comment

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

tested it out and it works perfectly!! AMAZINGGG

Copy link
Collaborator

@pragyakallanagoudar pragyakallanagoudar left a comment

Choose a reason for hiding this comment

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

oops just one note - can we discard the package-lock.json changes? i think this will cause merge conflicts for other PRs

Copy link
Collaborator

@pragyakallanagoudar pragyakallanagoudar left a comment

Choose a reason for hiding this comment

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

NVM - just got rid of them!

Copy link
Contributor

@ccatherinetan ccatherinetan left a comment

Choose a reason for hiding this comment

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

Looks great :D!

@pragyakallanagoudar pragyakallanagoudar changed the title [feat] Implements basic interest form [feat] implements basic interest form Oct 6, 2023
@pragyakallanagoudar pragyakallanagoudar merged commit 387ab55 into main Oct 6, 2023
2 checks passed
@pragyakallanagoudar pragyakallanagoudar deleted the alvaro/ijp-6-interest branch October 30, 2023 19:43
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.

3 participants