-
Notifications
You must be signed in to change notification settings - Fork 275
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/csv import #232
Feature/csv import #232
Conversation
@shouryan01 is attempting to deploy a commit to the Salgsmaskin Team on Vercel. A member of the Team first needs to authorize it. |
This looks very promising! Think this is the way to go, and we would also not need each person to added their bank. This will be a lot more dynamic β¨ |
Yeah @meglerhagen that was the idea! Now I had a question about mobile users: having that interface on a mobile sized screen is not a great experience; we'd need to build something from scratch for that use case. But I was thinking: we could just disable manual file uploads from mobile sized screens? I checked with Bank of America and Chase, and both their mobile apps don't have a way to export transactions like their desktop counterparts do. Therefore:
What do you think? If they want manual uploads we can just direct them to the website |
I agree with your approach. Lets disable it for mobile for now, even though we don't disable it its no problem. Minior fixes can the community also help with, while you do some heavy lifting ποΈ |
Description
Adds UI for uploading .csv files
What type of PR is this? (check all applicable)
Related Tickets & Documents
continues work on #196
Mobile & Desktop Screenshots/Recordings
Steps to QA
Added to documentation?
[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?