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

room join event is not triggered #291

Open
treeguard opened this issue Aug 11, 2020 · 4 comments
Open

room join event is not triggered #291

treeguard opened this issue Aug 11, 2020 · 4 comments

Comments

@treeguard
Copy link

Important:Please file the issue follow the template, or we won't help you to solve the problem.

0. Report Issue Guide

  1. Please run the following command and check whether the problem has been fixed:
npm install wechaty-puppet-padlus@next
  1. Please search in the issue first, and make sure your problem had not been reported before

1. Versions

  • What is your wechaty version?
    Answer:
    0.40.10
  • What is your wechaty-puppet-padplus version?
    Answer:
    0.6.3
  • What is your node version? (run node --version)
    Answer:
    v13.12.0
  • What os are you using
    Answer:
    Mac

2. Describe the bug

Here is my code

  room.on('join', async (room, inviteeList, inviter) => {
    log.info("bot room join")

I didn't see the listener function was triggered, but I saw it's triggered on donut puppet with the same piece of code. Is it because of it doesn't support async function?

3. To Reproduce

This part is very important: if you can not provide any reproduce steps, then the problem will be very hard to be recognized.

Steps to reproduce the behavior:

  1. run '...'
  2. ...
  3. ...

4. Expected behavior

Give a clear and concise description of what you expected to happen.

5. Actual behavior

If applicable, add screenshots to help explain your problem. But do not paste log screenshots here.

6. Full Output Logs

Set env WECHATY_LOG=silly in order to set log level to silly, then we can get the full log (If you dosen't set log env, log level is info as default, we cannot get the full log)

We need full log instead of log screenshot or log fragments!

Show Logs
$ WECHATY_LOG=silly node yourbot.js

Question: Paste your FULL(DO NOT ONLY PROVIDE FRAGMENTS) log messages
Answer:

7. Additional context

Add any other context about the problem here.

[bug]

@su-chang
Copy link
Member

Please upgrade [email protected], this version could handle it. Thank you.

@treeguard
Copy link
Author

treeguard commented Aug 11, 2020

@su-chang I have updated the pkg and use npm list wechaty-puppet-padplus to confirm but I still saw the issue, I printed out the object, the inviteeList is a single object not a list.

    log.info(`${JSON.stringify(inviteeList)}`)

@su-chang
Copy link
Member

I printed out the object, the inviteeList is a single object not a list.

Could you please share the log here?

@treeguard
Copy link
Author

I logged the function arguements, they are <invitee contact object, inviter contact object, timestamp string> which surprised me.

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

No branches or pull requests

2 participants