Remove invoice interstitial
- [x] The invoice modal doesn't add anything
- [x] Skip directly to the full page invoice view to approve
- [x] There needs to be a way to go back, without using the browser back button
- [x] Restore state appropriately (e.g. if 2 invoices are selected to be approved, they should remain approved)
- [x] Update invoice tests to be non-flakey
- [x] Add % equity split + amount to invoice on web (continue hiding on PDF/print)
Example back button:
I would like to pitch an alternate idea since we're working on this.
What if we made the invoice list page look like a mailbox with a preview section? That would make it more navigation-friendly and easier to skim through 🤔
https://github.com/user-attachments/assets/26971929-82d7-463d-8115-c95c061d7e66
Sharing v0 here
Let me know your thoughts everyone!
Interesting idea! Wdyt @ershad?
I like the idea too! I’ve made a few iterations on it on v0. We might also want to simplify the invoice view to reduce visual noise during review. @slavingia let me know if this approach works for you, and I can continue from here.
https://github.com/user-attachments/assets/c4850037-9346-4499-b4b2-ab7bf6b7e99f
Thinking more, I don't think we should invest in new UI/UX for a product only in maintenance mode. It'll be likely to introduce more bugs/issues.
@laugardie small update. We'd like to show the equity features in the page itself, as we show them in the modal that's going away. It should resemble the view that contractors have when creating the invoice with the details being in the bottom right and contain 20% as well as $1,000 (the absolute amount of equity being traded for).
We had this, then removed it, due to not wanting to show it on the printed invoice, but we can just do that with the print-specific styles.
@slavingia Made some designs here. Let me know your thoughts!
Requested access to the figma file
Commented! @slavingia would be great if you could have a look when you have a moment!
cc @ershad in case you're the right person to ping about this now :)