Correct import syntax but the Notifications file should be src/components/Common

Pass link param

need to pass the link param i.e. sendTokenNotification(link)

Put Notifications.js in src/components/Common , avoid using ../.. in imports; see pattern above

import {createTokenNotification} from `@components/Common/Notifications

Need to pass the link param

Move this file to the src/components/Common folder

Can remove these imports now that they are handled in a separate Notifications file

Need to pass in the balances, previousBalances params here

Remove this console.log statement

Remove. Use eTokenReceivedNotification for all token received cases.

Add a console.log statement here, as this can occasionally be useful when troubleshooting user reports.

Make sure the console.log statement includes which function called it to better assist in debugging.

For example:

console.log(Error passed to sendXecErrorNotification`, message)`

Improve this console.log statement like the one above.

Also -- why does this function accept an e parameter but sendXecErrorNotification does not?

Add in the function-referencing console.log statement as above.

In fact, we should have only one error notification function that also accepts an event as a parameter, then logs that to the console. Please investigate and implement this approach unless there is some reason it's impractical.

ok i see what was going on here. Please make the generalized error notification accept three parameters: error, message, event

then when you call it here, call it like errorNotification(error=e, message=message, event='Sending eToken')

Then you can console.log the event param in the function to assist in app debugging.

Uncomment this + make sure it's the generic error notification as discussed in above comments

Change name to xecReceivedNotification

be more specific with variable name than ev; use something like stringDescribingCallEvent

no need to spell this out, use errorNotification(e, message, 'Sending XEC')

same as earlier feedback, don't label parameters here

do a rebase to fix these prettier issues, we finally got the diff in that upgrades prettier for the whole web folder

same as earlier feedback, don't label parameters here

Remove unused imports, in this case notification

Remove unused imports, in this case notification

Remove unused imports, in this case notification