Skip to content

ref(issues): convert group sidebar to FC#48500

Merged
scefali merged 4 commits into
masterfrom
ref/convert-group-sidebar
May 8, 2023
Merged

ref(issues): convert group sidebar to FC#48500
scefali merged 4 commits into
masterfrom
ref/convert-group-sidebar

Conversation

@scefali

@scefali scefali commented May 3, 2023

Copy link
Copy Markdown
Contributor

Also moved the component

@scefali scefali requested review from a team and malwilley May 3, 2023 23:57
@scefali scefali requested a review from a team as a code owner May 3, 2023 23:57
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label May 3, 2023
@@ -0,0 +1,276 @@
import {Fragment, useCallback, useEffect, useState} from 'react';

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks like you didn't delete the other one

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@scefali scefali requested a review from evanpurkhiser May 4, 2023 00:13
Comment thread static/app/views/issueDetails/groupSidebar.tsx Outdated
Comment thread static/app/views/issueDetails/groupSidebar.tsx Outdated

trackAssign: OnAssignCallback = (type, _assignee, suggestedAssignee) => {
const {group, project, organization, location} = this.props;
export default function GroupSidebar({

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

very very small nit, but I am trying to move us off of default exports (very slowly).

Just something to keep in mind of the future.

RaduW pushed a commit that referenced this pull request May 8, 2023
@scefali scefali merged commit d51b67e into master May 8, 2023
@scefali scefali deleted the ref/convert-group-sidebar branch May 8, 2023 17:35
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 2, 2023
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.

2 participants