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

feat(feedback): show IconImage on feedback list if it has a screenshot #70109

Merged
merged 1 commit into from
May 3, 2024

Conversation

michellewzhang
Copy link
Member

@michellewzhang michellewzhang commented May 1, 2024

closes #67795
follow up to #70102 (the backend changes)

SCR-20240501-oexu

@michellewzhang michellewzhang requested a review from a team as a code owner May 1, 2024 23:14
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label May 1, 2024
michellewzhang added a commit that referenced this pull request May 2, 2024
…ck usecase (#70102)

relates to #67795

allow expansion of a new flag on an issue, `hasAttachments`, which will
specify whether a group has any event attachments. we'll use this in
user feedback to show a indicator when a feedback has an attachment
(e.g. screenshot).

context: the user feedback list shows small icon indicators when certain
properties are true (e.g. `hasReplay`, `hasLinkedError`) and we want to
add another one to this list: `hasAttachments` (aka `hasScreenshots`).
however, the list relies on having issue data only. we don't have access
to the attachments list just from the issue (we'd have to do a separate
api call on the frontend to get the list of attachments which is
unideal), so instead i'm adding an option to expand and directly put the
`hasAttachments` property onto the issue when requested.

frontend changes using this PR are here:
#70109
@michellewzhang michellewzhang merged commit abfe531 into master May 3, 2024
44 of 45 checks passed
@michellewzhang michellewzhang deleted the mz/screenshot-icon branch May 3, 2024 17:01
@github-actions github-actions bot locked and limited conversation to collaborators May 19, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Scope: Frontend Automatically applied to PRs that change frontend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Show <IconImage/> on feedback list if there is a screenshot
2 participants