Skip to content

feat: add getOperation([seq]) API #54

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

feat: add getOperation([seq]) API #54

wants to merge 10 commits into from

Conversation

@Nuhvi
Copy link

@Nuhvi Nuhvi commented May 25, 2022 •
edited
Loading

This is useful for example for watching operations on the bee

db.feed.on('download', (seq) => db.getOperation(seq))

Or

db.feed.on('append', () => db.feed.length > 1 && db.getOperation(db.feed.length - 1))

Also makes #23 a bit easier to achieve.

Nuhvi and others added 2 commits May 25, 2022 07:25
@Nuhvi
feat: add API
4456811 
same as createHistoryStream but without the overhead of creating then
reading a stream to get one operation
@Nuhvi
Merge branch 'master' into getoperation
456c67c
mafintosh
mafintosh reviewed Aug 16, 2022
View reviewed changes
@mafintosh
Copy link
Contributor

mafintosh commented Aug 16, 2022

@nazeh LGTM on the change requested and fixing the conflict

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mafintosh mafintosh mafintosh left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Nuhvi @mafintosh