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

Added Community-to-RST #223 #237

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

HappySR
Copy link

@HappySR HappySR commented Oct 16, 2024

  1. Deleted the .rev files from the community-team section.
  2. Copied the community-team to docs/ and then converted the .md files to .rst files.
  3. Updated the index.rst

Fixes #223

@HappySR
Copy link
Author

HappySR commented Oct 19, 2024

Created a PR named Added Community-to-RST #223 kindly review it, also kindly add the Hacktoberfest Accepted label if my PR works

Copy link
Member

@RCheesley RCheesley left a comment

Choose a reason for hiding this comment

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

Hi @HappySR thanks for making a start!

It looks like maybe you ran this file through a converter because there were a bunch of things not implemented correctly which are clearly documented in the issue like how to do headings and links. Please take a look in the future, as it takes us time to write those guidance notes ;)

I've fixed most of the fundamental issues like moving the files into the right place, creating the link files which were missing (you'd used inline links, which we don't use), and also fixing some of the heading structure which was using all kinds of heading levels and was completely missing in some places causing the RST checker to fail to build.

The PR should now build correctly but I have left you some Vale fixes to address before we can consider the PR for merging.

Sorry again for the delay, we've been a little swamped with contributions which means it's taken time to fix a couple of bugs with our Q/A pipeline and get these PRs reviewed.

Hopefully once these are fixed it'll be quick to merge! 🚀

Community dashboard
###################

The Community Dashboard uses the open-source :xref:`Savannah CRM` tool to import, analyze and visualize data from multiple sources in one central location.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
The Community Dashboard uses the open-source :xref:`Savannah CRM` tool to import, analyze and visualize data from multiple sources in one central location.
The Community Dashboard uses the open source :xref:`Savannah CRM` tool to import, analyze, and visualize data from multiple sources in one central location.


You can view the data from the last 90 days on the :xref:`Savannah Dashboard`.

Currently the dashboad imports:
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Currently the dashboad imports:
Currently the dashboard imports:


Currently the dashboad imports:

- All repositories listed under github.com/mautic
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- All repositories listed under github.com/mautic
- All repositories listed under ``github.com/mautic``

Currently the dashboad imports:

- All repositories listed under github.com/mautic
- Specified channels from mautic.slack.com
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- Specified channels from mautic.slack.com
- Specified channels from ``mautic.slack.com``


- All repositories listed under github.com/mautic
- Specified channels from mautic.slack.com
- All threads on forums.mautic.org
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- All threads on forums.mautic.org
- All threads on ``forum.mautic.org``

- All repositories listed under github.com/mautic
- Specified channels from mautic.slack.com
- All threads on forums.mautic.org
- All threads on reddit.com/r/mautic
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- All threads on reddit.com/r/mautic
- All threads on ``reddit.com/r/mautic``

- All threads on forums.mautic.org
- All threads on reddit.com/r/mautic
- All Stack Overflow posts tagged with Mautic
- Specified meetup.com groups (coming soon)
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- Specified meetup.com groups (coming soon)

This is no longer relevant (we don't use meetup.com any more) so I would remove this line al together.

- All Stack Overflow posts tagged with Mautic
- Specified meetup.com groups (coming soon)

The information is used to determine the activity levels of individuals and organizaitons within the community, which informs the monthly updates in Slack and the ranking of organizations in the :xref:`Partners' Directory`.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
The information is used to determine the activity levels of individuals and organizaitons within the community, which informs the monthly updates in Slack and the ranking of organizations in the :xref:`Partners' Directory`.
Mautic uses this information to determine the activity levels of individuals and organizations within the community, which informs the monthly updates in Slack and the ranking of organizations in the :xref:`Partners' Directory`.

*****************************************************

- Everyone who is enthusiastic enough
- As ambassadors: People in different countries who are already well-connected through other activities, and have a broad understanding of Mautic's facets (both from the user's and from the MarTech and even dev side, but no actual dev knowledge required)
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- As ambassadors: People in different countries who are already well-connected through other activities, and have a broad understanding of Mautic's facets (both from the user's and from the MarTech and even dev side, but no actual dev knowledge required)
- As ambassadors: People in different countries who are already well-connected through other activities, and have a broad understanding of Mautic's facets (both from the user's and from the MarTech and even developer side, but no actual developer knowledge required)

@favour-chibueze
Copy link
Contributor

@HappySR Pinging you in case you missed the above reviews

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.

Convert Community Team section to RST
3 participants