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

docs: Document integration request process. #14515

Closed
wants to merge 1 commit into from
Closed

docs: Document integration request process. #14515

wants to merge 1 commit into from

Conversation

majordwarf
Copy link
Collaborator

There was no proper documentation to guide user to request an integration.
The following changes documents the whole process and links it from the
/integrations/ page making it visible to the end-user.

Fixes #7935.

Integration Page:
image

Documentation Page:
image

@zulipbot
Copy link
Member

zulipbot commented Apr 8, 2020

Hello @zulip/server-integrations members, this pull request was labeled with the "area: integrations" label, so you may want to check it out!

Copy link
Member

@vinitS101 vinitS101 left a comment

Choose a reason for hiding this comment

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

Thanks for working on this, @majordwarf.

I left a few comments about some minor issues. :)

and [IFTTT](https://ifttt.com/) but there can be times where you don’t find an integration
or bot you need.

In such cases, you can request the integration by filling a feature request on our
Copy link
Member

Choose a reason for hiding this comment

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

I think this will be filing as in filing a complaint.

or bot you need.

In such cases, you can request the integration by filling a feature request on our
GitHub issue page. While filing an feature request please provide as much detail
Copy link
Member

Choose a reason for hiding this comment

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

This should probably be plural (issues), also, filing a

There was no proper documentation to guide user to request an integration.
The following changes documents the whole process and links it from the
`/integrations/` page making it visible to the end-user.

Fixes #7935.
@majordwarf
Copy link
Collaborator Author

@vinitS101 Made the changes requested! 👍

@majordwarf
Copy link
Collaborator Author

@timabbott This is ready to be merged.

@zulipbot
Copy link
Member

Heads up @majordwarf, we just merged some commits that conflict with the changes your made in this pull request! You can review this repository's recent commits to see where the conflicts occur. Please rebase your feature branch against the upstream/master branch and resolve your pull request's merge conflicts accordingly.

@timabbott
Copy link
Sponsor Member

Closing in favor of the active version at #17654.

@timabbott timabbott closed this Apr 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

/integrations-page: Add a request a new integration link.
4 participants