Skip to content
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

💻 Disable the teachers' survey #6147

Merged
merged 4 commits into from
Feb 7, 2025
Merged

💻 Disable the teachers' survey #6147

merged 4 commits into from
Feb 7, 2025

Conversation

boryanagoncharenko
Copy link
Collaborator

This PR temporarily disables the survey presented to teachers on the class overview page. This is the first step from the survey redesign process. This is why most of the code is not deleted but commented out. Ideally, it will be used again soon, when the redesign is done.

Fixes #6143

How to test

  • Run Hedy locally and log in as a teacher.
  • Go to For-Teachers and click on an existing class. When the class overview page opens, there should be no Survey link in the upper right corner (on the left of the 'Customize class' green button)
  • Create a new class and check that the Survey link is not displayed.

Copy link
Member

@jpelay jpelay left a comment

Choose a reason for hiding this comment

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

Great! Thanks for working on this one :)

Copy link
Contributor

mergify bot commented Feb 7, 2025

Thank you for contributing! Your pull request is now going on the merge train (choo choo! Do not click update from main anymore, and be sure to allow changes to be pushed to your fork).

Copy link
Contributor

mergify bot commented Feb 7, 2025

This pull request has been removed from the queue for the following reason: pull request manually updated.

The pull request #6147 has been manually updated

You should look at the reason for the failure and decide if the pull request needs to be fixed or if you want to requeue it.

If you want to requeue this pull request, you need to post a comment with the text: @mergifyio requeue

Copy link
Contributor

mergify bot commented Feb 7, 2025

Thank you for contributing! Your pull request is now going on the merge train (choo choo! Do not click update from main anymore, and be sure to allow changes to be pushed to your fork).

@mergify mergify bot merged commit 26a4f4d into main Feb 7, 2025
11 checks passed
@mergify mergify bot deleted the disable_survey_6143 branch February 7, 2025 00:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

💻 Disable the questionnaire
2 participants