Skip to content

Conversation

@ellie
Copy link
Member

@ellie ellie commented Jan 16, 2024

Yes I know I'm adding more coupling 😭

Plans are;

  1. Get this working fine with history
  2. When introducing new stores (less legacy, less tangled), figure out a nice design
  3. Port history onto new design

@vercel
Copy link

vercel bot commented Jan 16, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
atuin-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 16, 2024 6:12pm

@ellie ellie force-pushed the ellie/history-delete branch from 683a70f to 03905ad Compare January 16, 2024 17:58
@ellie ellie marked this pull request as ready for review January 16, 2024 17:58
@ellie
Copy link
Member Author

ellie commented Jan 16, 2024

All of this can be enabled with

[sync]
records = true

In config. Once enabled, please don't disable again. The old sync may not react well, and old deleted records may "magically" re-appear (they'll only be deleted in the new store, for now)

@ellie
Copy link
Member Author

ellie commented Jan 16, 2024

will give a once over and merge in the morning, I'm 😴

@ellie ellie merged commit d9317a1 into main Jan 17, 2024
@ellie ellie deleted the ellie/history-delete branch January 17, 2024 09: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.

2 participants