Skip to content

Conversation

@cnbrenci
Copy link
Contributor

@cnbrenci cnbrenci commented Dec 3, 2025

This PR does a couple things:

  • Add an experiment to do the new permissions work behind
  • Replace CanEnableAITutor with using the experiment flag & remove CanEnableAITutor
  • Rename/reposition the tutor settings in teacher dashboard and put it behind the experiment

Links

Testing story

Verified no change to the teacher dashboard unless the experiment is enabled:

experiment disabled:
image

experiment enabled:
image

Deployment strategy

everything currently behind experiment flag

Follow-up work

  • The rest of the implementation for the actual content of the AI Settings tab

@cnbrenci
Copy link
Contributor Author

cnbrenci commented Dec 3, 2025

@codex review

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Copy link
Contributor

@drizco drizco left a comment

Choose a reason for hiding this comment

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

lgtm! love the updated comments!

@cnbrenci cnbrenci merged commit 878b46a into staging-next Dec 4, 2025
3 checks passed
@cnbrenci cnbrenci deleted the cassi/tutor/permissions/reposition-navigation branch December 4, 2025 18:14
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.

2 participants