Skip to content

Commit

Permalink
made some minor adjustments like removing unused vars
Browse files Browse the repository at this point in the history
  • Loading branch information
ydahal1 committed Jan 18, 2024
1 parent 724399c commit bc7fad0
Show file tree
Hide file tree
Showing 4 changed files with 2 additions and 10 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -219,6 +219,7 @@ function JobMonitoring() {

//Set newly added job monitoring to jobMonitorings
const responseData = await response.json();

if (editingData.isEditing) {
const updatedJobMonitorings = jobMonitorings.map((jobMonitoring) => {
if (jobMonitoring.id === responseData.id) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -27,11 +27,6 @@ module.exports = {
allowNull: false,
type: Sequelize.BOOLEAN,
},
// isApproved: {
// allowNull: true,
// type: Sequelize.BOOLEAN,
// },

approvalStatus: {
allowNull: false,
type: Sequelize.ENUM("Approved", "Rejected", "Pending"),
Expand Down
4 changes: 0 additions & 4 deletions server/models/jobMonitoring.js
Original file line number Diff line number Diff line change
Expand Up @@ -22,10 +22,6 @@ module.exports = (sequelize, DataTypes) => {
allowNull: false,
type: DataTypes.BOOLEAN,
},
// isApproved: {
// allowNull: true,
// type: DataTypes.BOOLEAN,
// },
approvalStatus:{
allowNull: false,
type: DataTypes.ENUM("Approved", "Rejected", "Pending"),
Expand Down
2 changes: 1 addition & 1 deletion server/routes/jobmonitoring/read.js
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,7 @@ router.post(
}
});

// Patch a single job monitoring with express validator
// Patch a single job monitoring
router.patch( "/",
[
body("id").isUUID().withMessage("ID must be a valid UUID"),
Expand Down

0 comments on commit bc7fad0

Please sign in to comment.