integration-docs: Update the SVN integration doc.#871
Conversation
Niloth-p
left a comment
There was a problem hiding this comment.
Thank you for the review! Updated as recommended.
I've not made any changes related to the existence of the target channel, as I'm unsure how to proceed.
I've switched to using backticks instead of double quotes for channel names, and added the prefix # in front of channel names (because that's the convention in the Help docs - example), based on your review comment on another doc PR.
|
The screenshot looks like really outdated Zulip UI. |
|
@laurynmm Do you have any further feedback? |
That should now be updated by the example screenshots PR that was merged yesterday. 😄 However, I just noticed that this actually requires a prior PR that adds the macro |
|
After reading over the updates, my only question is do we want to link to the Subversion project anywhere? Either in the main description or in a Related documentation section? Or both? |
|
Added a Related documentation section with a link to We could replace the macro that requires a prerequisite PR to be merged in the other repo, with the text to get this PR merged immediately. And later apply the macro to all the affected integrations. Let me know. |
|
Okay. I think the macro question has been dealt with since zulip/zulip#36240 was merged to the |
|
Merged, thanks @Niloth-p! |



Pre-requisite PR zulip/zulip#36240 adds the macro
install-requirements.mdthat this doc uses.Screenshots
SVN doc link

Self-review checklist
(variable names, code reuse, readability, etc.).
Communicate decisions, questions, and potential concerns.
Individual commits are ready for review (see commit discipline).
Completed manual review and testing of the following: