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

Feature/196 csv import #249

Merged
merged 7 commits into from
Jun 8, 2024
Merged

Conversation

shouryan01
Copy link
Collaborator

Description

This PR implements file import during onboarding and pushes it to the database

What type of PR is this? (check all applicable)

  • 💡 Feature
  • 🐛 Bug Fix
  • 📝 Documentation Update
  • 🎨 Style
  • 🧑‍💻 Code Refactor
  • 🔥 Performance Improvements
  • ✅ Test
  • 🤖 Build
  • 🔁 CI
  • 📦 Chore (Release)
  • ⏩ Revert

Related Tickets & Documents

Adds features to #196, specifically the file import from bank exports

Mobile & Desktop Screenshots/Recordings

image
image
image
image
image

image

Steps to QA

  1. Go to /onboarding
  2. Click through to the file import step
  3. Use your bank export file or use the example file provided in the files
  4. The file contents are now in neon

Added to documentation?

  • 📜 README.md
  • 🙅 no documentation needed

[optional] Are there any post-deployment tasks we need to perform?

[optional] What gif best describes this PR or how it makes you feel?

Copy link

vercel bot commented May 28, 2024

@shouryan01 is attempting to deploy a commit to the Salgsmaskin Team on Vercel.

A member of the Team first needs to authorize it.

@shouryan01
Copy link
Collaborator Author

The currency is set to be "USD" as a default, it is annoying to enter that every time especially if someone is using manual imports exclusively, the currency should be part of account settings,

Currently using the first account from accounts table

Copy link

vercel bot commented May 28, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
badget ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 28, 2024 0:10am

Copy link
Owner

@Codehagen Codehagen left a comment

Choose a reason for hiding this comment

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

LGTM ✨

@Codehagen Codehagen merged commit 8a5bac7 into Codehagen:main Jun 8, 2024
1 check failed
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