-
Notifications
You must be signed in to change notification settings - Fork 14
Conversation
…e into submitCounter
… checkIfAvailable.js
It is weird that we do need |
I fixed a merge conflict for |
f2/server.js
Outdated
console.log('Warning: redirecting request to CAFC') | ||
res.redirect( | ||
req.subdomains.includes('signalez') | ||
? 'http://www.antifraudcentre-centreantifraude.ca/report-signalez-fra.htm' | ||
: 'http://www.antifraudcentre-centreantifraude.ca/report-signalez-eng.htm', | ||
) | ||
} else { | ||
console.log('number of submissions: ' + availableData.numberOfSubmissions) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can 🔥 these two new console logs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
f2/src/utils/saveRecord.js
Outdated
(date.getMonth() > 8 ? date.getMonth() + 1 : '0' + (date.getMonth() + 1)) + | ||
'/' + | ||
date.getFullYear() | ||
console.log(currentDate) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🔥 this log
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
f2/src/utils/saveRecord.js
Outdated
} else { | ||
db.close() | ||
numberofReports = result.length | ||
console.log('number of reports submitted: ' + numberofReports) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🔥 this log
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ran locally 👍 Awesome work!!
Just a couple debugging console.log()
's to get rid of, and a merge conflict and we'll be ready to go!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good stuff! 🚀
* Minor copy change (typo) (#1639) * extract data from formidable manually (#1641) * fix: upgrade mongodb from 3.5.2 to 3.5.3 (#1632) * fix: upgrade react-scripts from 3.3.1 to 3.4.0 (#1642) * Fix for Logs in another Azure Subscription (#1619) * Updating IMAGE_NAME reference (#1597) * change GNCC to GNC3 (#1655) * use redirect for equifax links (#1651) * clear file input after alert (#1662) * fix keyboard nav (#1660) * add moneyLost.methodOther field to analyst report (#1647) * more detailed p and text stories (#1643) * added stuff to sb docs (#1671) * added getting started page to storybook (#1661) * FIX CORS for HTTPS (#1665) (#1669) * clean up logging a bit (#1663) * tweak skiplink style (#1658) * fix target area for upload button (#1653) * change "go back" button's height to the same height as "cancel report" button (#1666) * added Messages component. (#1644) * fix: upgrade async from 3.1.1 to 3.2.0 (#1680) * Reports a day counter (#1664) * Form validation - bump (#1510) * added helmet in server.js (#1674) * Combine analyst emails (#1628) * Prod version number (#1656) * Fixed Field, added container and layout to sb (#1675) * Get email addresses from ldap query (#1682) * restrict file number and size on server (#1648) * remove submit button if already submitted (#1683) * Fix for Continuous Deployment (#1690) * Validation for location fields (#1689) * cypress pack initial commit (#1247)
Fixes #1616
Description
Any new packages installed?
Required followup work
Checklist: