Skip to content
This repository was archived by the owner on Sep 19, 2024. It is now read-only.

Config/context overridden by another webhook#801

Draft
Sadaf-A wants to merge 13 commits intoubiquity:developmentfrom
Sadaf-A:feature/config-context
Draft

Config/context overridden by another webhook#801
Sadaf-A wants to merge 13 commits intoubiquity:developmentfrom
Sadaf-A:feature/config-context

Conversation

@Sadaf-A
Copy link
Contributor

@Sadaf-A Sadaf-A commented Sep 23, 2023

Resolves #732

@Sadaf-A Sadaf-A requested a review from cuddle-dev as a code owner September 23, 2023 10:58
@netlify
Copy link

netlify bot commented Sep 23, 2023

Deploy Preview for ubiquibot-staging ready!

Name Link
🔨 Latest commit bee6ac6
🔍 Latest deploy log https://app.netlify.com/sites/ubiquibot-staging/deploys/65377bcfb6774e0008b38071
😎 Deploy Preview https://deploy-preview-801--ubiquibot-staging.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Member

@0x4007 0x4007 left a comment

Choose a reason for hiding this comment

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

Needs to pass CI before you request a review.

@0x4007 0x4007 marked this pull request as draft September 23, 2023 14:46
@0x4007 0x4007 requested a review from whilefoo September 23, 2023 14:47
@0x4007
Copy link
Member

0x4007 commented Sep 23, 2023

I guess this is for you @whilefoo

Copy link
Collaborator

@whilefoo whilefoo left a comment

Choose a reason for hiding this comment

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

your code is failing because you removed botConfig variable but everywhere in the code it's still used

@Sadaf-A Sadaf-A marked this pull request as ready for review October 24, 2023 08:11
@Sadaf-A Sadaf-A marked this pull request as draft October 24, 2023 08:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Config/context overridden by another webhook

4 participants