Skip to content

Note lack of feature parity with RITlug site (closes #87) #92

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

Merged
merged 1 commit into from
Feb 26, 2020

Conversation

justwheel
Copy link
Member

This commit adds a note to the README that this website does not have
full feature parity with the RITlug site, and links to issue #87 where
there is more specific details on the differences.

Closes #87.

@justwheel justwheel added improvement Improves on something that already exists documentation Improvements or additions to documentation labels Feb 26, 2020
@justwheel justwheel requested a review from a team February 26, 2020 19:19
@justwheel justwheel self-assigned this Feb 26, 2020
@justwheel justwheel requested review from kennedy and kevinassogba and removed request for a team February 26, 2020 19:19
Copy link
Member

@ct-martin ct-martin left a comment

Choose a reason for hiding this comment

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

Something to the effect of "however, the codebase has since diverged based on our needs" might be more discreet if you want the same effect without potentially implying one site is better than the other.

You're also missing a comma after "however"

Otherwise, LGTM; I like the alt/title text on the link

Copy link
Member

@whenbellstoll whenbellstoll left a comment

Choose a reason for hiding this comment

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

Good to note. Approved. Closes #92

This commit adds a note to the README that this website does not have
full feature parity with the RITlug site, and links to issue #87 where
there is more specific details on the differences.

Closes #87.

Signed-off-by: Justin W. Flory <[email protected]>
@justwheel justwheel force-pushed the change/87-ritlug-disclaimer branch from 2646ba3 to d87fd2a Compare February 26, 2020 21:39
@justwheel justwheel merged commit 0b5f538 into master Feb 26, 2020
@justwheel justwheel deleted the change/87-ritlug-disclaimer branch February 26, 2020 21:40
Copy link
Member

@kevinassogba kevinassogba left a comment

Choose a reason for hiding this comment

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

Looks useful to me. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation improvement Improves on something that already exists
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Be more explicit that this site does not have feature parity with RITlug's
4 participants