Skip to content

Conversation

@TobyCyan
Copy link
Contributor

Part of #12048
Part of #13324

Outline of Solution

  • Migrated InstructorFeedbackResultsPage to its SQL version.
  • Wrote the test.json for InstructorFeedbackReportPageE2ETest.java.

@github-actions
Copy link

Hi @TobyCyan, thank you for your interest in contributing to TEAMMATES!
However, your PR does not appear to follow our contribution guidelines:

  • Title must start with the issue number the PR is fixing in square brackets, e.g. [#<issue-number>]

Please address the above before we proceed to review your PR.

@TobyCyan TobyCyan changed the title [12048] Migrate E2E Tests for InstructorFeedbackReportPage [#12048] Migrate E2E Tests for InstructorFeedbackReportPage Jan 19, 2026
@TobyCyan TobyCyan marked this pull request as draft January 19, 2026 17:21
@mingyuancode
Copy link
Contributor

Please pass the e2e

Copy link
Contributor

@mingyuancode mingyuancode left a comment

Choose a reason for hiding this comment

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

Please change it

@samuelfangjw samuelfangjw added the c.Task Other non-user-facing works, e.g. refactoring, adding tests label Jan 26, 2026
@samuelfangjw samuelfangjw added this to the V9.0.0-beta.7 milestone Jan 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c.Task Other non-user-facing works, e.g. refactoring, adding tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants