Skip to content

Commit

Permalink
Merge pull request #5688 from PR4NJ41/dev28-Native-Assignment-Deletion
Browse files Browse the repository at this point in the history
Implemented Native confirm component instead of JavaScript confirm instance
  • Loading branch information
ragesoss committed Mar 14, 2024
2 parents 4ee3a44 + ead1ae7 commit ab15ec8
Showing 1 changed file with 8 additions and 3 deletions.
11 changes: 8 additions & 3 deletions app/assets/javascripts/components/articles/available_article.jsx
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import React from 'react';
import PropTypes from 'prop-types';
import { useDispatch } from 'react-redux';

import { initiateConfirm } from '~/app/assets/javascripts/actions/confirm_actions';
import CourseUtils from '../../utils/course_utils.js';
import { deleteAssignment, claimAssignment } from '../../actions/assignment_actions.js';

Expand Down Expand Up @@ -37,8 +37,13 @@ export const AvailableArticle = ({ assignment, current_user, course, selectable
role: 0
};

if (!confirm(I18n.t('assignments.confirm_deletion'))) { return; }
return dispatch(deleteAssignment(assignmentObj));
const confirmMessage = I18n.t('assignments.confirm_deletion');

const onConfirm = () => {
dispatch(deleteAssignment(assignmentObj));
};

dispatch(initiateConfirm({ confirmMessage, onConfirm }));
};

const className = 'assignment';
Expand Down

0 comments on commit ab15ec8

Please sign in to comment.