Skip to content

Commit

Permalink
resolved the PR comments
Browse files Browse the repository at this point in the history
  • Loading branch information
Pratiyushkumar committed Nov 2, 2023
1 parent 5fc0768 commit fdcaad1
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 4 deletions.
4 changes: 2 additions & 2 deletions app/components/task/holder.hbs
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@
data-test-task-status-select
id='task-update'
{{on 'change' this.onStatusChange}}
disabled={{eq @task.status 'DONE'}}
disabled={{eq @task.status this.TASK_KEYS.DONE}}
>
{{#each this.removedTaskStatus as |taskStatus|}}
<option
Expand All @@ -52,7 +52,7 @@
</div>

{{#if @dev}}
{{#if (not-eq @task.status 'DONE')}}
{{#if (not-eq @task.status this.TASK_KEYS.DONE)}}
<button
class='task-card__extensionForm-button'
data-test-task-extensionForm-button
Expand Down
1 change: 0 additions & 1 deletion app/components/task/holder.js
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@ export default class TasksHolderComponent extends Component {
@tracked status = this.args.task.status;
@tracked extensionFormOpened = false;
@tracked isLoading = false;
@tracked disable = false;
queryParams = ['dev'];

TASK_KEYS = TASK_KEYS;
Expand Down
2 changes: 1 addition & 1 deletion app/constants/tasks.js
Original file line number Diff line number Diff line change
Expand Up @@ -135,7 +135,7 @@ const TABS_TASK_STATUS_LIST = [
},
{
displayLabel: 'Done',
key: COMPLETED,
key: DONE,
},
{
displayLabel: 'Verified',
Expand Down

0 comments on commit fdcaad1

Please sign in to comment.