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

Updating ReadMe Documentation: #1004

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

KaiKitKobold
Copy link

Since the initial Read.me documentation was lacking with helpful sources and information about the project, and sources relating to the project, I've updated the Read.Me, to include other links to the project and sections of content relating to various aspects of the project.

Please let me know if this does not fit expectations; I can easily update it to improve it.

Committing this will update the Read.Me, Including a table of contents, descriptions of the project, and links to documentation relating to the project.
@KaiKitKobold
Copy link
Author

I Will Add that this is a first draft, and I expect this Commit to be flawed Initially anyway; I'll keep iterating until it gets a thumbs up from various Maintainers.

This Commit fixes the issue where the about-us could be seen as pro-piracy, it has been changed to a much more descriptive representation of the app. 
And reinforces our stance against Piracy.
@KaiKitKobold
Copy link
Author

Read.Me has been updated. Please let me know of any potential errors and what needs to be changed or updated.

Copy link
Contributor

@Blatzar Blatzar left a comment

Choose a reason for hiding this comment

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

Generally good, but needs a few fixes


[![Discord](https://invidget.switchblade.xyz/5Hus6fM)](https://discord.gg/5Hus6fM)
Copy link
Contributor

Choose a reason for hiding this comment

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

Please keep the discord somewhere in the top so people dont miss it 🙏

README.md Outdated

## About us:

**CloudStream can be defined in many ways. However, it's best described as a media center that prioritizes and emphasizes complete freedom and total flexibility between users and developers.**
Copy link
Contributor

Choose a reason for hiding this comment

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

No need for the first sentence

Suggested change
**CloudStream can be defined in many ways. However, it's best described as a media center that prioritizes and emphasizes complete freedom and total flexibility between users and developers.**
**CloudStream is best described as a media center that prioritizes and emphasizes complete freedom and flexibility for users and developers.**

README.md Outdated

**CloudStream can be defined in many ways. However, it's best described as a media center that prioritizes and emphasizes complete freedom and total flexibility between users and developers.**

A more descriptive example of Cloudstream is that it's an extension-based multimedia player with tracking support; an example of extensions for CloudStream includes Support for:
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
A more descriptive example of Cloudstream is that it's an extension-based multimedia player with tracking support; an example of extensions for CloudStream includes Support for:
CloudStream is an extension-based multimedia player with tracking support. There are extensions to view videos from:

README.md Outdated
+ [nginx](https://nginx.org/)
+ And more...

To reiterate, **we do not support Piracy;** for more details about our stance on the DMCA and EUCD, you can read about it on our Organization: [ReCloudStream](https://github.com/recloudstream)
Copy link
Contributor

Choose a reason for hiding this comment

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

This is the first iteration

Suggested change
To reiterate, **we do not support Piracy;** for more details about our stance on the DMCA and EUCD, you can read about it on our Organization: [ReCloudStream](https://github.com/recloudstream)
**We do not support Piracy.** For more details about our stance on the DMCA and EUCD, you can read about it on our Organization: [recloudstream](https://github.com/recloudstream)


## Installation:

Our documentation provides the steps to Install Cloudstream. The steps are simple and extensively customizable for your streaming needs!
Copy link
Contributor

Choose a reason for hiding this comment

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

The steps themselves are not customizable

Suggested change
Our documentation provides the steps to Install Cloudstream. The steps are simple and extensively customizable for your streaming needs!
Our documentation provides the steps to install and configure CloudStream for your streaming needs.

README.md Outdated
<a id="issues"></a>

### Issues:
While we **actively** accept issues and pull requests, we do not have any syntaxial rules; for issue contributions, however, it is heavily appreciated if your issue includes the following:
Copy link
Contributor

Choose a reason for hiding this comment

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

We do have syntaxial rules, by the templates when making issues: https://github.com/recloudstream/cloudstream/issues/new/choose

This section is therefore probably better for linking to the issues, e.g:
https://github.com/recloudstream/cloudstream/issues/new?assignees=&labels=bug&projects=&template=application-bug.yml

https://github.com/recloudstream/cloudstream/issues/new?assignees=&labels=enhancement&projects=&template=feature-request.yml

And describing that for bug reports we want as much info as possible just to make it possible to replicate the bug in order to fix it.

And for feature requests its very important to check that its not already requested to prevent spam.

README.md Outdated

### Extensions:

**Further details on creating extensions for CloudStream are found on our website. **
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
**Further details on creating extensions for CloudStream are found on our website. **
**Further details on creating extensions for CloudStream are found on our website.**
If you need help with development please come ask in our discord.

README.md Outdated Show resolved Hide resolved
Quick update to READ.ME With changes specified by maintainer @Blatzar
@KaiKitKobold
Copy link
Author

Hello @Blatzar! (I'm Sorry for accidentally deleting one of your suggestions; I didn't know what the Resolve conversation actually did until I pressed it.)
I have updated my commit with the changes you specified above. Please let me know if anything else needs to be changed. I look forward to your feedback.

@KaiKitKobold
Copy link
Author

Oh Whoops, It seems that I missed a step, I apologize, I'll get started working on it right away.

Updated Read.me to include suggestions from both Green and Yapper. 
Mainly changed wording to be more professional on our stance on piracy and changed the issues tab.
Fixed broken links for #Enhancment, and #Bug Reports.
@KaiKitKobold
Copy link
Author

Hello @Blatzar, I've updated the ReadME to include your and Green's suggestions; if you wish for anything in the issues header or anything else in the project to be fixed, please let me know.

Removed Social in the Further Sources, since it doesn't host the discord server link anymore
@fire-light42
Copy link
Collaborator

Any update on this @Blatzar

@KaiKitKobold
Copy link
Author

@Blatzar, I am commenting to let you know that I'm still available and ready to update this project if needed.

<a id="issues"></a>

### Issues:
While we **actively** accept issues and pull requests; We do require you fill out an [template](https://github.com/recloudstream/cloudstream/issues/new/choose) for issues, these include the following:
Copy link
Contributor

Choose a reason for hiding this comment

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

I would change punctuation:

Suggested change
While we **actively** accept issues and pull requests; We do require you fill out an [template](https://github.com/recloudstream/cloudstream/issues/new/choose) for issues, these include the following:
While we **actively** accept issues and pull requests, we do require you fill out an [template](https://github.com/recloudstream/cloudstream/issues/new/choose) for issues. These include the following:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants