Skip to content

Conversation

levadadenys
Copy link
Contributor

@levadadenys levadadenys commented Oct 22, 2025

[AITT-1254] Remove getting started prompts for teachers with an active section

  • feat: enhance default suggested prompts based on teacher's sections and curriculum
2025-10-22.15.49.28.mov
Знімок екрана 2025-10-22 о 15 53 06 Знімок екрана 2025-10-22 о 15 52 21 Знімок екрана 2025-10-22 о 15 50 42

Links

Testing story

Deployment strategy

Follow-up work

Privacy

Security

Caching

PR Creation Checklist:

  • Tests provide adequate coverage
  • Privacy impacts have been documented
  • Security impacts have been documented
  • Code is well-commented
  • New features are translatable or updates will not break translations
  • Relevant documentation has been added or updated
  • User impact is well-understood and desirable
  • Follow-up work items (including potential tech debt) are tracked and linked

@levadadenys levadadenys requested review from a team, cearachew, Copilot and etaderhold October 22, 2025 12:58
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR enhances the default suggested prompts in the AI Differentiation chat by conditionally hiding certain getting-started prompts for teachers who already have active sections with curriculum and students. The changes implement logic to check a teacher's section status and filter out prompts like "Get Started", "Create Section", and "Suggest Curriculum" when they're no longer relevant.

  • Adds helper function to filter default suggested prompts based on teacher's section status
  • Introduces checks for teacher sections, curriculum assignment, and student enrollment
  • Updates default prompt logic to provide context-appropriate suggestions

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment on lines +80 to +87
if (
label === SUGGEST_CURRICULUM_PROMPT.label &&
teacherHasSectionWithCurriculum
) {
return false;
}

return true;
Copy link
Contributor

Choose a reason for hiding this comment

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

Nit: This can be simplified to return label === SUGGEST_CURRICULUM_PROMPT.label && teacherHasSectionWithCurriculum

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yep, but I keep as is intentionally, so that it's easier to read and understand what's going on

@tess323
Copy link

tess323 commented Oct 22, 2025

🎉

Copy link
Contributor

@cearachew cearachew left a comment

Choose a reason for hiding this comment

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

LGTM

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.

4 participants