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

Send PPro feedback to support inbox #3567

Merged
merged 8 commits into from
Nov 26, 2024
Merged

Send PPro feedback to support inbox #3567

merged 8 commits into from
Nov 26, 2024

Conversation

quanganhdo
Copy link
Member

Task/Issue URL: https://app.asana.com/0/1200019156869587/1207999338171708/f
Tech Design URL:
CC:

Description:

Integrates with backend to submit issue reports to support inbox.

Optional E2E tests:

  • Run PIR E2E tests
    Check this to run the Personal Information Removal end to end tests. If updating CCF, or any PIR related code, tick this.

Steps to test this PR:

  1. Go through the unified feedback flow
  2. When choosing to submit an issue, there'll be a new optional email field
  3. Filling in the form with a valid email would trigger the backend integration

Definition of Done:


Internal references:

Pull Request Review Checklist
Software Engineering Expectations
Technical Design Template
Pull Request Documentation

@quanganhdo quanganhdo changed the title Anh/ppro/support inbox Send PPro feedback to support inbox Nov 18, 2024
@quanganhdo quanganhdo requested a review from samsymons November 18, 2024 17:32
Copy link
Collaborator

@samsymons samsymons left a comment

Choose a reason for hiding this comment

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

LGTM 🎉

@quanganhdo quanganhdo merged commit 6c9724c into main Nov 26, 2024
20 checks passed
@quanganhdo quanganhdo deleted the anh/ppro/support-inbox branch November 26, 2024 14:54
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