Zulip node can not update user for "is_guest"

Questions
1

I tested zulip node with option update user for is_guest, but it seems failed.

2

Hey @cmdntd987,

Did you get an error? If yes, can you please share what error you received? I am changing this to a question since it seems more of a bug than a feature request :slight_smile:

3

It just says “success”, but test member still is “member”
This is my test:

kevPC1UjYt
kevPC1UjYt830×437 13 KB

4

Thanks for sharing the information. It does seem like a bug. I am assigning this to @RicardoE105.

1 Like
6

Thanks. And I do not see where to post bug topic in this forum like before?

7

It looks like is_admin ad is_guess were replace with role

image
image1628×788 78.8 KB

8

Could we change boolean option by a value field?
It seems set by value is more flexible and more safe for next upgrade from zulip

9

Sadly not. The solution is to add the role parameter.

10

Could I wait for next upgrade of this node?

11

Got added. However, I could not test it because I kept getting permission issues. It’s probably something with my setup. Can you please and confirm that it works?

12

Yes, I will test it when upgraded.
Thank you very much.

13

We cannot merge to master until it is adequately tested. Either you test it locally and let me know, or provide me with test credentials (privately) so that I can test it.

14

How could I update this branch for n8n?

15

That depends. How are you running n8n?

16

I am running on docker, manage by portainer
Database: SQLite

17

As mentioned on the GitHub issue did I create the docker image n8nio/n8n:PR-2336-feature-zulip-extended which contains the code of the PR that it can be tested.

1 Like
18

It does not work, still success without “is_guest”
I do not know why it said error with credentials when notify “success”

This is log from portainer

  cause: Error: Node does not have any credentials set!
      at new NodeOperationError (/data/packages/workflow/src/NodeErrors.ts:207:12)
      at getCredentials (/data/packages/core/src/NodeExecuteFunctions.ts:943:11)
      at Object.getCredentials (/data/packages/core/src/NodeExecuteFunctions.ts:1801:12)
      at Object.zulipApiRequest (/data/packages/nodes-base/nodes/Zulip/GenericFunctions.ts:18:33)
      at Object.getUsers (/data/packages/nodes-base/nodes/Zulip/Zulip.node.ts:135:47)
      at LoadNodeParameterOptions.getOptions (/data/packages/core/src/LoadNodeParameterOptions.ts:123:51)
      at /data/packages/cli/src/Server.ts:1101:30
      at processTicksAndRejections (internal/process/task_queues.js:93:5)
      at /data/packages/cli/src/ResponseHelper.ts:144:17,
  node: {
    parameters: {
      resource: 'message',
      operation: 'sendPrivate',
      to: '',
      content: ''
    },
    name: 'Temp-Node',
    type: 'n8n-nodes-base.zulip',
    typeVersion: 1,
    position: [ 0, 0 ]
  },
  timestamp: 1634741466535
}

chrome_xGg3kXnU6X

chrome_ngxFgK2q9q

19

Did you try the role parameter?

20

Nope. How could I do it for update user option?

21

Hi, Could you test to complete this issue, please?