Skip to content

Hide the "delete" option from Channel Edit page#2910

Merged
micahscopes merged 1 commit intolearningequality:hotfixesfrom
sairina:hide-delete
Feb 8, 2021
Merged

Hide the "delete" option from Channel Edit page#2910
micahscopes merged 1 commit intolearningequality:hotfixesfrom
sairina:hide-delete

Conversation

@sairina
Copy link
Copy Markdown
Contributor

@sairina sairina commented Feb 5, 2021

Description

Because the functionality of "Delete channel" from the Channel Edit page is currently not working, we are just hiding this option from the view for the time being.

Issue Addressed (if applicable)

Addresses #2894 (but does not fix the issue - this is only temporary)

Before/After Screenshots (if applicable)

Before After
2021-02-01 14 19 11 Screen Shot 2021-02-04 at 7 54 06 PM

Steps to Test

  • Login to Studio
  • Click on an existing channel from the Channel List page
  • Click on the menu at the upper right hand corner (three dots next to "PUBLISH") of the Channel Editing page
  • Notice that "Delete channel" is no longer an option in the dropdown menu

Does this introduce any tech-debt items?

This is temporary, and the functionality needs to be fixed in the future.

Checklist

Delete any items that don't apply

  • Is the code clean and well-commented?
  • Are there tests for this change?
  • Are views organized into pages, components, and layouts directories as described in the docs?

@rtibbles rtibbles added this to the Post Release Stabilization milestone Feb 6, 2021
Copy link
Copy Markdown
Contributor

@micahscopes micahscopes left a comment

Choose a reason for hiding this comment

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

Looks good!

@micahscopes micahscopes merged commit bb1d8e6 into learningequality:hotfixes Feb 8, 2021
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.

3 participants