Skip to content

Conversation

@madebydavid
Copy link
Member

Description

This is to resolve #76 so that people can easily see which account they are using

What's changed?

  • If the user is logged in then the label for the "Logged In" row in the status table includes the user's email address in bold
  • Made an asciinema recording of this and included this in the docs

Demo

asciicast

@madebydavid madebydavid marked this pull request as ready for review August 21, 2025 10:32
@madebydavid madebydavid requested a review from Copilot August 21, 2025 10:33
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR enhances the shipthis status command to display the current user's email address alongside the login status, making it easier for users to identify which account they're using. This addresses issue #76 by providing better visibility into the active authentication state.

  • Enhanced status display to include email address when user is logged in
  • Updated documentation with new asciinema recording showing the enhanced output

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
src/commands/status.tsx Added email extraction and conditional display in status table
docs/status.md Updated asciinema recording link to demonstrate new functionality

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@madebydavid
Copy link
Member Author

Thanks Copilot

@madebydavid madebydavid merged commit 4dd685c into main Aug 21, 2025
@madebydavid madebydavid deleted the 76-feature-display-my-current-shipthis-account-email-in-the-shipthis-status-info branch August 21, 2025 10:41
@madebydavid madebydavid changed the title Showing current email address in shipthis status output feat: showing current email address in shipthis status output Aug 28, 2025
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.

Feature: Display my current shipthis account email in the shipthis status info

2 participants