Skip to content

Commit

Permalink
fix: Delete scheduled change request (#4437)
Browse files Browse the repository at this point in the history
  • Loading branch information
kyle-ssg authored Jul 31, 2024
1 parent f03b479 commit 233ce50
Showing 1 changed file with 6 additions and 7 deletions.
13 changes: 6 additions & 7 deletions frontend/web/components/pages/ChangeRequestPage.js
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ import DiffChangeRequest from 'components/diff/DiffChangeRequest'
import NewVersionWarning from 'components/NewVersionWarning'
import { mergeChangeSets } from 'common/services/useChangeRequest'
import WarningMessage from 'components/WarningMessage'
import ErrorMessage from 'components/ErrorMessage';
import ErrorMessage from 'components/ErrorMessage'

const ChangeRequestsPage = class extends Component {
static displayName = 'ChangeRequestsPage'
Expand Down Expand Up @@ -177,8 +177,8 @@ const ChangeRequestsPage = class extends Component {
getScheduledDate = (changeRequest) => {
return changeRequest.environment_feature_versions.length > 0
? moment(changeRequest.environment_feature_versions[0].live_from)
: changeRequest?.change_sets
? changeRequest?.change_sets?.[0]?.live_from
: changeRequest?.change_sets?.[0]?.live_from
? moment(changeRequest?.change_sets?.[0]?.live_from)
: moment(changeRequest.feature_states[0].live_from)
}

Expand Down Expand Up @@ -363,16 +363,15 @@ const ChangeRequestsPage = class extends Component {
/>
<PageTitle
cta={
(!changeRequest?.committed_at ||
this.getScheduledDate(changeRequest) > moment()) && (
(!changeRequest?.committed_at || isScheduled) && (
<Row>
<Button
theme='secondary'
onClick={this.deleteChangeRequest}
>
Delete
</Button>
{!isVersioned && (
{!isVersioned && !changeRequest?.committedAt && (
<Button
onClick={() =>
this.editChangeRequest(
Expand Down Expand Up @@ -567,7 +566,7 @@ const ChangeRequestsPage = class extends Component {
</div>
)
)}
<ErrorMessage error={ChangeRequestStore.error}/>
<ErrorMessage error={ChangeRequestStore.error} />
{changeRequest.committed_at ? (
<div className='mr-2 font-weight-medium'>
Committed at{' '}
Expand Down

0 comments on commit 233ce50

Please sign in to comment.