Skip to content

Conversation

@carl-adams-planet
Copy link
Contributor

Pull in latest planet-auth library to improve the UX around expired sessions.

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 improves the UX around expired sessions by updating the planet-auth library and implementing a new ensure_initialized() method that handles session state management more intelligently.

  • Updates planet-auth dependency to version 2.3.0a1759946524 for improved expired session handling
  • Adds new ensure_initialized() method that automatically handles session refresh and user prompts only when necessary
  • Updates example code to demonstrate the improved session management pattern

Reviewed Changes

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

File Description
pyproject.toml Updates planet-auth dependency to new pre-release version
planet/auth.py Adds ensure_initialized() method and improves type annotations
examples/auth-session-management/*.py Updates examples to use new ensure_initialized() pattern

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

Copy link
Contributor

@asonnenschein asonnenschein left a comment

Choose a reason for hiding this comment

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

LGTM! The detailed docstring for ensure_initialized() is super helpful!

@carl-adams-planet carl-adams-planet changed the title DRAFT: CG-1904 improve expired session ux CG-1904 improve expired session ux Oct 20, 2025
@carl-adams-planet carl-adams-planet merged commit 0cea292 into main Oct 20, 2025
9 checks passed
@carl-adams-planet carl-adams-planet deleted the carl/CG-1904--improve-expired-session-ux branch October 20, 2025 18:58
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.

3 participants