Skip to content

Conversation

@xmok
Copy link
Contributor

@xmok xmok commented Jan 29, 2026

Description

  • In Invoices, Show "Last Sent" and "Last Viewed"

Screencast

N/A - minor change

Checklist

@raycastbot raycastbot added extension fix / improvement Label for PRs with extension's fix improvements extension: wave Issues related to the wave extension platform: macOS OP is author The OP of the PR is the author of the extension labels Jan 29, 2026
@raycastbot
Copy link
Collaborator

Thank you for the update! 🎉

We're currently experiencing a high volume of incoming requests. As a result, the initial review may take up to 10-15 business days.

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Jan 29, 2026

Greptile Overview

Greptile Summary

This PR adds "Last Sent" and "Last Viewed" information to the Wave invoice details view. The implementation includes:

  • Added GraphQL fields (lastSentAt, lastSentVia, lastViewedAt) to fetch the data
  • Created InvoiceSendMethod enum to type the send method values
  • Added formatInvoiceDate utility function for displaying timestamps with time
  • Updated the invoice detail metadata to display the new fields

Issues Found:

  • The "Last sent" display logic has incorrect ordering that could show malformed text when an invoice was sent (bug on line 167)

Confidence Score: 3/5

  • This PR has a logical bug that needs fixing before merge
  • The PR adds a useful feature with proper TypeScript typing and GraphQL integration, but the display logic for "Last sent" has a bug that will show incorrect text for sent invoices
  • extensions/wave/src/manage-wave.tsx requires attention for the display logic bug

Important Files Changed

Filename Overview
extensions/wave/src/lib/types.ts Added InvoiceSendMethod enum and new fields to Invoice type
extensions/wave/src/lib/utils.ts Added formatInvoiceDate function for timestamp formatting with time
extensions/wave/src/manage-wave.tsx Displayed "Last sent" and "Last viewed by customer" in invoice metadata, but the "Last sent" display logic has an issue

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

3 files reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

@pernielsentikaer pernielsentikaer self-assigned this Jan 29, 2026
Copy link
Collaborator

@pernielsentikaer pernielsentikaer left a comment

Choose a reason for hiding this comment

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

Looks good to me, approved 🔥

@raycastbot raycastbot merged commit ecdffdc into raycast:main Jan 29, 2026
@github-actions
Copy link
Contributor

Published to the Raycast Store:
https://raycast.com/xmok/wave

@raycastbot
Copy link
Collaborator

🎉 🎉 🎉

We've rewarded your Raycast account with some credits. You will soon be able to exchange them for some swag.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

extension fix / improvement Label for PRs with extension's fix improvements extension: wave Issues related to the wave extension OP is author The OP of the PR is the author of the extension platform: macOS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants