Skip to content

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 <git@jwf.io>
@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

5 participants