Skip to content

Ship the registered EVE client ID as the sign-in default - #58

Closed
Fazorath wants to merge 1 commit into
mainfrom
issue-57-default-client-id
Closed

Fazorath wants to merge 1 commit into
mainfrom
issue-57-default-client-id

Conversation

@Fazorath

Copy link
Copy Markdown
Owner

Closes #57

_client_id() now returns the built-in public client ID (PKCE, no secret) when EVE_CLIENT_ID is unset or empty; a set value overrides it. Tests cover default, empty and override. README and run-windows skill note the variable is optional.

Tests: 72 pass; the 5 tkinter-dependent modules can't import on this host (same without the change).

@Fazorath

Copy link
Copy Markdown
Owner Author

Superseded by #59 (rebased on main; force-push was blocked by a hook).

@Fazorath Fazorath closed this Oct 28, 2026
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.

Ship the registered EVE client ID as the default so sign-in works with nothing set

1 participant