Feature: Add Community Onboarding Steps #3056
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Because: * The current community information is buried in Foundations * These lesson is long and needs to be split into smaller articles * Every Discord community member needs to be aware of the rules * Every Discord community member needs to know how to ask a question This Commit: * Adds CommunityOnboardingComponent and test * Adds community controller and route * Adds community_onboarded and community_onboarding_steps to user model * Adds update_onboarding_steps to user's controller & route * Adds tests for user_spec and community#show * Update user's ability to update onboarding steps * Translate dashboard_steps markdown to html * Adds partials for github_link and form for checkbox * Darkened background of app's checkbox * Replace ODIN_CHAT_URL with community_path * Update welcome email templates * Removes 2 lessons: Asking For Help & Join the Odin Community
|
This has probably been discussed but with this change the intention is only those registered for a TOP account will now have access to the discord? |
|
@CouchofTomato Good question! Yes, it has been discussed at team meetings and also in this thread with an earlier version of screenshots. |
|
Change Requested: Create a new discord invite link, so that moving forward we can track what invite links people are using. |
ChargrilledChook
added
the
Type: Enhancement
KevinMulhern
requested review from
I3uckwheat
and removed request for
I3uckwheat
|
Closing due to the team deciding to not implement this feature. Instead we will be making a community page feature. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Complete the following REQUIRED checkboxes:
keyword: brief description of changeformat, using one of the following keywords:Feature- adds new or amends existing user-facing behaviourChore- changes that have no user-facing value, refactors, dependency bumps, etcFix- bug fixesComplete the following checkbox ONLY IF it is applicable to your PR. You can complete it later if it is not currently applicable:
1. Because:
Related to TheOdinProject/top-meta#66
2. This PR:
3. Additional Information:
When this PR is ready to merge (or right after), we will need to make an announcement in Discord because the behavior of the "Community" link is changing drastically.
In addition, after it is merged we will need to change links throughout the curriculum and in Discord's rule channel because this PR changes the id's to more readable format.