flexile icon indicating copy to clipboard operation
flexile copied to clipboard

Remove invoice interstitial

Open slavingia opened this issue 1 month ago • 10 comments

  • [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:

Image

slavingia avatar Nov 05 '25 16:11 slavingia

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!

sm17p avatar Nov 05 '25 17:11 sm17p

Interesting idea! Wdyt @ershad?

slavingia avatar Nov 05 '25 17:11 slavingia

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

laugardie avatar Nov 06 '25 11:11 laugardie

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.

slavingia avatar Nov 06 '25 14:11 slavingia

@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 avatar Nov 12 '25 20:11 slavingia

@slavingia Made some designs here. Let me know your thoughts!

laugardie avatar Nov 13 '25 12:11 laugardie

Requested access to the figma file

MayaRainer avatar Nov 14 '25 16:11 MayaRainer

Commented! @slavingia would be great if you could have a look when you have a moment!

MayaRainer avatar Nov 19 '25 17:11 MayaRainer

cc @ershad in case you're the right person to ping about this now :)

MayaRainer avatar Nov 24 '25 14:11 MayaRainer

I also liked @sm17p's idea, but we can't do that right now because of the reasons @slavingia mentioned here.

Replied in figma.

ershad avatar Nov 29 '25 19:11 ershad