Skip to content

Prevent: TypeError: Cannot read properties of undefined (reading 'id')#52

Open
marius-cornescu wants to merge 1 commit into
travisghansen:masterfrom
marius-cornescu:marius-cornescu-patch-1
Open

Prevent: TypeError: Cannot read properties of undefined (reading 'id')#52
marius-cornescu wants to merge 1 commit into
travisghansen:masterfrom
marius-cornescu:marius-cornescu-patch-1

Conversation

@marius-cornescu

Copy link
Copy Markdown

Prevent exceptions like:

TypeError: Cannot read properties of undefined (reading 'id')
at recurse (/data/node_modules/node-red-contrib-slack/slackpost.js:568:50)
at recurse (/data/node_modules/node-red-contrib-slack/slackpost.js:540:17)
at SlackConfig.dressResponseMessage (/data/node_modules/node-red-contrib-slack/slackpost.js:589:11)
at /data/node_modules/node-red-contrib-slack/slackpost.js:1306:43
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

image

Prevent exceptions like:

TypeError: Cannot read properties of undefined (reading 'id')
    at recurse (/data/node_modules/node-red-contrib-slack/slackpost.js:568:50)
    at recurse (/data/node_modules/node-red-contrib-slack/slackpost.js:540:17)
    at SlackConfig.dressResponseMessage (/data/node_modules/node-red-contrib-slack/slackpost.js:589:11)
    at /data/node_modules/node-red-contrib-slack/slackpost.js:1306:43
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
@marius-cornescu

Copy link
Copy Markdown
Author

teams can be missing from the state:
image

@travisghansen

Copy link
Copy Markdown
Owner

Great! I will merge soon and publish a new release.

@marius-cornescu

Copy link
Copy Markdown
Author

@travisghansen - any updates ?

@bellomimarc

Copy link
Copy Markdown

I have the same issue. any updates?

@bellomimarc

Copy link
Copy Markdown

@travisghansen @marius-cornescu I start a new open source project in my company https://www.npmjs.com/package/@qomodome/node-red-slack

PRs are welcome!

@travisghansen

Copy link
Copy Markdown
Owner

Is it compatible in api to this project? I am happy to allow additional owners to this gh and/or the npm package if you prefer to go that route.

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.

3 participants