Skip to content

Add total history count to the index API#1102

Merged
ellie merged 1 commit into
mainfrom
ellie/county-count
Jul 14, 2023
Merged

Add total history count to the index API#1102
ellie merged 1 commit into
mainfrom
ellie/county-count

Conversation

@ellie

@ellie ellie commented Jul 14, 2023

Copy link
Copy Markdown
Member

Thought it would be fun to collect some cool stats, maybe put them on atuin.sh.

Thought it would be fun to collect some cool stats, maybe put them on
atuin.sh.
@ellie ellie requested a review from conradludgate July 14, 2023 18:31
@vercel

vercel Bot commented Jul 14, 2023

Copy link
Copy Markdown

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

Name Status Preview Updated (UTC)
atuin ✅ Ready (Inspect) Visit Preview Jul 14, 2023 6:31pm

.fetch_optional(&self.pool)
.await
.map_err(fix_error)?
.unwrap_or((0,));

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

None = no history send up at all yet :(

@conradludgate conradludgate left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Sure, why not

@ellie ellie merged commit 5786155 into main Jul 14, 2023
@ellie ellie deleted the ellie/county-count branch July 14, 2023 19:44
ealap pushed a commit to ealap/atuin that referenced this pull request Jul 18, 2023
Thought it would be fun to collect some cool stats, maybe put them on
atuin.sh.
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