refactor: Replace create-one-time-access-token script with in-app functionality (v1.0) #170

New Issue

AtHeartEngineer · 2025-07-08T08:41:30-04:00

AtHeartEngineer commented

2025-07-08 08:41:30 -04:00

Originally created by @ItalyPaleAle on 5/17/2025

As discussed here: https://github.com/pocket-id/pocket-id/pull/533#discussion_r2093286087

Removes the create-one-time-access-token.sh script and embeds the functionality within the Go binary.

It can now be invokes as pocket-id one-time-access-token <user name or email>.

This helps because:

Reduces complexity and improves maintenability because there's no duplicate functionality in the shell script
Ensures the shell script works well with SQLite even after changing the connection string format

*Originally created by @ItalyPaleAle on 5/17/2025* As discussed here: https://github.com/pocket-id/pocket-id/pull/533#discussion_r2093286087 Removes the create-one-time-access-token.sh script and embeds the functionality within the Go binary. It can now be invokes as `pocket-id one-time-access-token <user name or email>`. This helps because: 1. Reduces complexity and improves maintenability because there's no duplicate functionality in the shell script 2. Ensures the shell script works well with SQLite even after changing the connection string format

AtHeartEngineer closed this issue

2025-07-08 08:41:31 -04:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: github/pocket-id#170