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

[#12048] Migrate SubmitFeedbackResponseAction's tests #12760

Conversation

FergusMok
Copy link
Contributor

Part of #12048

3rd out of the 3 PRs to migrate SubmitFeedbackResponseAction (#12732 and #12720)

Remove unnecessary comments

Remove unnecessary comments

Save progress

Add draft

Add draft
…ommentsCascade

Revert unnecessary changes, add tests

Fix linting and tests

Add changes

Revert changes

Revert changes

Add tests

Add lint changes
Copy link

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

  • Description must reference the issue number the PR is fixing, e.g. Fixes #<issue-number> (or Part of #<issue-number> if the PR does not address the issue fully)

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

@FergusMok FergusMok added the s.Ongoing The PR is being worked on by the author(s) label Feb 19, 2024
@NicolasCwy NicolasCwy changed the base branch from master to v9-migration February 20, 2024 03:32
@FergusMok FergusMok force-pushed the 12048-migrate-submit-feedbackresponse-action2 branch from 461568f to 8c31ec6 Compare February 24, 2024 01:48
@FergusMok FergusMok closed this Apr 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
s.Ongoing The PR is being worked on by the author(s)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant