tisonkun commented on code in PR #414:
URL: https://github.com/apache/pulsar-site/pull/414#discussion_r1107879685


##########
src/pages/community/DiscussionPlatforms.tsx:
##########
@@ -0,0 +1,195 @@
+import React from "react";
+import useBaseUrl from "@docusaurus/useBaseUrl";
+import DiscussionPlatform, {
+  DiscussionPlatformProps,
+} from "./DiscussionPlatform";
+import s from "./DiscussionPlatforms.module.css";
+
+const DiscussionPlatforms: React.FC = () => {
+  const platforms: DiscussionPlatformProps[] = [
+    {
+      name: "User Mailing List",
+      description: (
+        <div>
+          General mailing list for user-related discussions.
+          <br />
+          <a
+            href="https://lists.apache.org/[email protected]";
+            target="_blank"
+          >
+            Access the archives
+          </a>
+        </div>
+      ),
+      actions: [
+        {
+          id: "subscribe",
+          text: "Subscribe",
+          href: "[email protected]",
+          type: "primary",
+        },
+        {
+          id: "unsubscribe",
+          text: "Unsubscribe",
+          href: "[email protected]",
+          type: "normal",
+        },
+      ],
+      logoSrc: useBaseUrl("/img/mailing-list.svg"),
+    },
+    {
+      name: "Developer Mailing List",
+      description: (
+        <div>
+          Questions and discussions related to Pulsar development.
+          <br />
+          <a
+            href="https://lists.apache.org/[email protected]";
+            target="_blank"
+          >
+            Access the archives
+          </a>
+        </div>
+      ),
+      actions: [
+        {
+          id: "subscribe",
+          text: "Subscribe",
+          href: "[email protected]",
+          type: "primary",
+        },
+        {
+          id: "unsubscribe",
+          text: "Unsubscribe",
+          href: "[email protected]",
+          type: "normal",
+        },
+      ],
+      logoSrc: useBaseUrl("/img/mailing-list.svg"),
+    },
+    {
+      name: "Discussions at GitHub",
+      description: (
+        <div>
+          A good place to ask any question, bring an idea or get support. 
Especially if you are not
+          friends with mailing lists.
+        </div>
+      ),
+      actions: [
+        {
+          id: "new-discussion",
+          text: "New discussion",
+          href: "https://github.com/apache/pulsar/discussions/new";,
+          type: "primary",
+          isExternal: true,
+        },
+        {
+          id: "open",
+          text: "Browse",
+          href: "https://github.com/apache/pulsar/discussions";,
+          type: "normal",
+          isExternal: true,
+        },
+      ],
+      logoSrc: useBaseUrl("/img/github-mark.svg"),
+    },
+    {
+      name: "Stack Overflow",
+      description: (
+        <span>
+          For technical questions, we ask that you post them to Stack Overflow
+          using the tag{" "}
+          <code style={{ whiteSpace: "nowrap" }}>apache-pulsar</code>.
+        </span>
+      ),
+      actions: [
+        {
+          id: "as",
+          text: "Ask question",
+          href: "https://stackoverflow.com/questions/ask?tags=apache-pulsar";,
+          type: "primary",
+          isExternal: true,
+        },
+        {
+          id: "browse",
+          text: "Browse",
+          href: "https://stackoverflow.com/questions/tagged/apache-pulsar";,
+          type: "normal",
+          isExternal: true,
+        },
+      ],
+      logoSrc: useBaseUrl("/img/stackoverflow-logo.svg"),
+    },
+    {
+      name: "Slack",
+      description: (
+        <>
+          Use it for instant messaging and real-time discussions.
+          <br />
+          <br />
+          Keep in mind that asking questions in Slack makes it harder to find
+          the answers later, due to Slack history isn&apos;t indexable by 
search
+          engines like Google.
+        </>
+      ),
+      actions: [
+        {
+          id: "open",
+          text: "Open",
+          href: "https://apache-pulsar.slack.com/";,
+          type: "primary",
+          isExternal: true,
+        },
+        {
+          id: "sign-up",
+          text: "Sign-up",
+          href: 
"https://communityinviter.com/apps/apache-pulsar/apache-pulsar";,
+          type: "normal",
+          isExternal: true,
+        },
+      ],
+      logoSrc: useBaseUrl("/img/Slack_Mark.svg"),
+    },
+    {
+      name: "WeChat",

Review Comment:
   Actually, WeChat Official Account is not a place for discussions but for 
subscriptions. Can we host this info in a new section? It's a bit out of the 
scope of this patch but the current layout is not quite in balance (personal 
bias, though :))



##########
src/pages/community/DiscussionPlatforms.tsx:
##########
@@ -0,0 +1,195 @@
+import React from "react";
+import useBaseUrl from "@docusaurus/useBaseUrl";
+import DiscussionPlatform, {
+  DiscussionPlatformProps,
+} from "./DiscussionPlatform";
+import s from "./DiscussionPlatforms.module.css";
+
+const DiscussionPlatforms: React.FC = () => {
+  const platforms: DiscussionPlatformProps[] = [
+    {
+      name: "User Mailing List",
+      description: (
+        <div>
+          General mailing list for user-related discussions.
+          <br />
+          <a
+            href="https://lists.apache.org/[email protected]";
+            target="_blank"
+          >
+            Access the archives
+          </a>
+        </div>
+      ),
+      actions: [
+        {
+          id: "subscribe",
+          text: "Subscribe",
+          href: "[email protected]",
+          type: "primary",
+        },
+        {
+          id: "unsubscribe",
+          text: "Unsubscribe",
+          href: "[email protected]",
+          type: "normal",
+        },
+      ],
+      logoSrc: useBaseUrl("/img/mailing-list.svg"),
+    },
+    {
+      name: "Developer Mailing List",
+      description: (
+        <div>
+          Questions and discussions related to Pulsar development.
+          <br />
+          <a
+            href="https://lists.apache.org/[email protected]";
+            target="_blank"
+          >
+            Access the archives
+          </a>
+        </div>
+      ),
+      actions: [
+        {
+          id: "subscribe",
+          text: "Subscribe",
+          href: "[email protected]",
+          type: "primary",
+        },
+        {
+          id: "unsubscribe",
+          text: "Unsubscribe",
+          href: "[email protected]",
+          type: "normal",
+        },
+      ],
+      logoSrc: useBaseUrl("/img/mailing-list.svg"),
+    },
+    {
+      name: "Discussions at GitHub",
+      description: (
+        <div>
+          A good place to ask any question, bring an idea or get support. 
Especially if you are not
+          friends with mailing lists.
+        </div>
+      ),
+      actions: [
+        {
+          id: "new-discussion",
+          text: "New discussion",
+          href: "https://github.com/apache/pulsar/discussions/new";,
+          type: "primary",
+          isExternal: true,
+        },
+        {
+          id: "open",
+          text: "Browse",
+          href: "https://github.com/apache/pulsar/discussions";,
+          type: "normal",
+          isExternal: true,
+        },
+      ],
+      logoSrc: useBaseUrl("/img/github-mark.svg"),
+    },
+    {
+      name: "Stack Overflow",
+      description: (
+        <span>
+          For technical questions, we ask that you post them to Stack Overflow
+          using the tag{" "}
+          <code style={{ whiteSpace: "nowrap" }}>apache-pulsar</code>.
+        </span>
+      ),
+      actions: [
+        {
+          id: "as",
+          text: "Ask question",
+          href: "https://stackoverflow.com/questions/ask?tags=apache-pulsar";,
+          type: "primary",
+          isExternal: true,
+        },
+        {
+          id: "browse",
+          text: "Browse",
+          href: "https://stackoverflow.com/questions/tagged/apache-pulsar";,
+          type: "normal",
+          isExternal: true,
+        },
+      ],
+      logoSrc: useBaseUrl("/img/stackoverflow-logo.svg"),
+    },
+    {
+      name: "Slack",
+      description: (
+        <>
+          Use it for instant messaging and real-time discussions.
+          <br />
+          <br />
+          Keep in mind that asking questions in Slack makes it harder to find
+          the answers later, due to Slack history isn&apos;t indexable by 
search
+          engines like Google.
+        </>
+      ),
+      actions: [
+        {
+          id: "open",
+          text: "Open",
+          href: "https://apache-pulsar.slack.com/";,
+          type: "primary",
+          isExternal: true,
+        },
+        {
+          id: "sign-up",
+          text: "Sign-up",
+          href: 
"https://communityinviter.com/apps/apache-pulsar/apache-pulsar";,
+          type: "normal",
+          isExternal: true,
+        },

Review Comment:
   Perhaps name the buttons as "Sign-up" and "Launch Slack" and switch the 
order so that align with GitHub Discussions.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to