Skip to content
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.

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

Added resteem functionality to Responder #18

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

TwinkieWInkie
Copy link
Collaborator

@TwinkieWInkie TwinkieWInkie commented Apr 15, 2018

Untested as of now, not expecting any errors as the implementation is correct.
close #6

@TwinkieWInkie TwinkieWInkie changed the title Added resteem functionality to Responder Added resteem functionality to Responder #6 Apr 15, 2018
@TwinkieWInkie TwinkieWInkie changed the title Added resteem functionality to Responder #6 Added resteem functionality to Responder Apr 15, 2018
@p0o
Copy link
Owner

p0o commented Apr 16, 2018

Thank you very much. I will test it soon by the meantime can you please change your indention from tabs to 2 space which is the convention for this codebase.

@TwinkieWInkie
Copy link
Collaborator Author

TwinkieWInkie commented Apr 16, 2018 via email

Copy link
Owner

@p0o p0o left a comment

Choose a reason for hiding this comment

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

There are some issues with the code, I tried debugging it for a short period but even with fixing these issues the bot is not resteeming. Not sure what is the problem maybe you test yourself and check.

const wif = this.postingKey || this.activeKey;

const action = JSON.stringify(['reblog'], {
account: this.responserUsername,
Copy link
Owner

Choose a reason for hiding this comment

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

typo in "responder"

wif,
[],
[this.responderUsername],
'follow',
Copy link
Owner

Choose a reason for hiding this comment

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

Why the id for transaction is "follow" ? I don't remember exactly but I think it should be a random string like a uuid. since the new commit in node from the second execution it returns this error:

Unhandled rejection RPCError: Assert Exception:_callbacks.find( txid ) == _callbacks.end(): Transaction is a duplicate
    at new RPCError (/Users/p0o/dev/steem-bot-tests/steem-bot/node_modules/steem/lib/api/transports/http.js:39:106)
    at /Users/p0o/dev/steem-bot-tests/steem-bot/node_modules/steem/lib/api/transports/http.js:74:13
    at <anonymous>
    at process._tickCallback (internal/process/next_tick.js:188:7)

@p0o
Copy link
Owner

p0o commented May 13, 2018

@TwinkieWInkie Hey, you still want to finish this?

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

Successfully merging this pull request may close these issues.

Resteem others' posts with this API?
2 participants