Don't show the "Leave section" button to students in OAuth sections #17047
Conversation
|
This component still shows the |
|
My understanding (which may be imperfect) is that SectionsTable will go away after the section-flow-2017 experiment becomes enabled. Presumably that implies we'll want to make this same sort of change there? |
| onClick={this.onLeave.bind(this, section.code, section.name)} | ||
| color={Button.ButtonColor.gray} | ||
| /> | ||
| {!/^(C|G)-/.test(section.code) && |
There was a problem hiding this comment.
I might make a helper method called something like isOauthSection(section) that can include a comment about what this regex is doing/why we can only leave oauth sections.
|
Apparently SectionsTable will not go away completely, as it will still be used by the |
|
@poorvasingal does this mean we need to hide the |
+cc @poorvasingal