Skip to content

Add versioned URL updates with history/restore + clippy reverse sort fixes#97

Draft
Copilot wants to merge 3 commits into
mainfrom
copilot/implement-version-controlled-url-updates
Draft

Add versioned URL updates with history/restore + clippy reverse sort fixes#97
Copilot wants to merge 3 commits into
mainfrom
copilot/implement-version-controlled-url-updates

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 18, 2026

Add support for updating a short link’s destination while retaining a full, queryable history, without impacting hot-path reads. Ensure transactional integrity, cache invalidation, and resolve clippy’s reverse-sort warnings.

  • Data models & storage API

    • Added URL history entry + update/restore request models and storage trait methods.
  • Storage implementations (Postgres/SQLite)

    • Created url_history table + index.
    • Implemented transactional update/restore flows and history retrieval.
    • Cached storage now invalidates on update/restore.
  • API surface

    • Added update, history, and restore handlers/routes with owner/admin authorization.
  • Clippy cleanup

    • Replaced reverse sort_by with sort_by_key(Reverse(...)).

Example (reverse sort idiom used):

result.sort_by_key(|item| std::cmp::Reverse(item.visit_count));

Copilot AI and others added 3 commits May 18, 2026 12:36
Agent-Logs-Url: https://github.com/BTreeMap/Lynx/sessions/e0ccef91-c537-4877-9350-a64831e2350d

Co-authored-by: MinecraftFuns <25814618+MinecraftFuns@users.noreply.github.com>
Agent-Logs-Url: https://github.com/BTreeMap/Lynx/sessions/e0ccef91-c537-4877-9350-a64831e2350d

Co-authored-by: MinecraftFuns <25814618+MinecraftFuns@users.noreply.github.com>
Agent-Logs-Url: https://github.com/BTreeMap/Lynx/sessions/e0ccef91-c537-4877-9350-a64831e2350d

Co-authored-by: MinecraftFuns <25814618+MinecraftFuns@users.noreply.github.com>
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.

2 participants