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

[NV-2960] 🚀 Feature: Add Zulip Chat Provider #4380

Closed
2 tasks done
Cliftonz opened this issue Oct 3, 2023 · 8 comments · Fixed by #4457
Closed
2 tasks done

[NV-2960] 🚀 Feature: Add Zulip Chat Provider #4380

Cliftonz opened this issue Oct 3, 2023 · 8 comments · Fixed by #4457
Assignees
Labels
good first issue Good for newcomers
Milestone

Comments

@Cliftonz
Copy link
Contributor

Cliftonz commented Oct 3, 2023

🔖 Feature description

Use the Zulip docs and the zulip js package to create a chat provider while following Novu's guide to a new provider

🎤 Why is this feature needed ?

give more providers to the community

✌️ How do you aim to achieve this?

community contribution

🔄️ Additional Information

No response

👀 Have you spent some time to check if this feature request has been raised before?

  • I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

Are you willing to submit PR?

None

NV-2960

@Cliftonz Cliftonz added the good first issue Good for newcomers label Oct 3, 2023
@Cliftonz Cliftonz changed the title 🚀 Feature: Add Zulip Chat Provider [NV-2960] 🚀 Feature: Add Zulip Chat Provider Oct 3, 2023
@michaldziuba03
Copy link
Contributor

I would like to implement this

@Yash-sudo-web
Copy link
Contributor

@michaldziuba03 are you working on this issue?, if not @Cliftonz can i work on this?

@Ank61
Copy link

Ank61 commented Oct 4, 2023

Hey @michaldziuba03 , @Cliftonz! I would like to contribute to this issue.
Thanks

@michaldziuba03
Copy link
Contributor

@Yash-sudo-web @Ank61

relax, I work on this

@Yash-sudo-web
Copy link
Contributor

@Cliftonz if this issue is not being worked upon, i already have a PR that adds this feature, let me know if we can reopen that PR

@Cliftonz
Copy link
Contributor Author

Cliftonz commented Oct 7, 2023

@michaldziuba03 have you completed this PR?
I know that I assigned other items to you.

If you have not then I will reassign and reopen @Yash-sudo-web PR for this.

@michaldziuba03
Copy link
Contributor

@Cliftonz
today or tomorrow you can expect PR for this issue

@Cliftonz
Copy link
Contributor Author

Cliftonz commented Oct 7, 2023

@michaldziuba03 Sounds good, I do not like to push but our policy is a draft PR/ PR needs have progress within three days or we will need to reassign.

@Cliftonz Cliftonz modified the milestone: Cycle 29 Oct 7, 2023
@Cliftonz Cliftonz modified the milestone: Cycle 30 Oct 23, 2023
@Cliftonz Cliftonz added this to the Cycle 30 milestone Oct 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants