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(alerts): Add project incident aler to proj/issue stream page - NATIVE-216 & NATIVE-217 #28726

Merged
merged 9 commits into from
Sep 23, 2021

Conversation

priscilawebdev
Copy link
Member

@priscilawebdev priscilawebdev commented Sep 21, 2021

Previews:

Project Details
image

Issue Stream

image

ps:

Closes: NATIVE-216 and NATIVE-217

@@ -561,6 +561,9 @@ def serialize(self, obj, attrs, user):
"hasAccess": attrs["has_access"],
"dateCreated": obj.date_added,
"environments": attrs["environments"],
"eventProcessing": {
Copy link
Member Author

Choose a reason for hiding this comment

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

this will be changed later by the backend team.

Copy link
Contributor

@flub flub left a comment

Choose a reason for hiding this comment

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

LGTM!

@markstory
Copy link
Member

That stack of alerts at the top of the issue view is awkward. Do we have plans to revisit that?

@priscilawebdev priscilawebdev marked this pull request as draft September 22, 2021 09:51
@priscilawebdev priscilawebdev changed the title feat(alerts): Add project incident aler to proj/issue details page - NATIVE-216 & NATIVE-217 feat(alerts): Add project incident aler to proj/issue stream page - NATIVE-216 & NATIVE-217 Sep 23, 2021
@priscilawebdev
Copy link
Member Author

priscilawebdev commented Sep 23, 2021

thanks for the feedback @markstory, we've decided to move this alert from the issue details page to the stream page (temp solution), which makes more sense but is still not ideal. @Jesse-Box is already working on a better solution that will be implemented in a separate PR

@priscilawebdev priscilawebdev marked this pull request as ready for review September 23, 2021 09:26
Copy link
Member

@matejminar matejminar left a comment

Choose a reason for hiding this comment

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

code looks good to me

Copy link
Contributor

@Jesse-Box Jesse-Box left a comment

Choose a reason for hiding this comment

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

🚢 it

@priscilawebdev priscilawebdev merged commit a1dba54 into master Sep 23, 2021
@priscilawebdev priscilawebdev deleted the feat/add-project-incident-alert branch September 23, 2021 11:15
vuluongj20 pushed a commit that referenced this pull request Sep 30, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Oct 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants