Next

Closes #1687 and #1640

added 1 commit

• 582b5a7b - notifs dont seem to be working properly on sandbox, need to figure that out

Compare with previous version

Commented out after hook for now as with it in things cannot be debugged (cypress outputs that the test failed in the after hook if it fails anywhere).

Notifications don't seem to be working for me on the sandbox.

comment on user 2s post from user 1. User 2 receives no notification.

Don't disable tests if they are not working. That's a terrible habit to get into. We'll get the notifications fixed on sandbox when Mark is available tomorrow.

Oh I know. This is just a WIP. It's just this hook here: !492 (diffs)

The reason is that if the test fails, it will always say that the error is in the after hook and not provide the correct failing section (when its actually in the tests). Because I was testing how it ran in the CI, I needed to leave it out so that I could actually get any outputted error.

added 6 commits

• 582b5a7b...14a72a8a - 5 commits from branch master
• a8f3f663 - Merge remote-tracking branch 'origin/master' into feat/notification-e2e-1640

Compare with previous version

added 69 commits

• a8f3f663...d69d2bd0 - 68 commits from branch master
• d4941cdc - Merge branch 'master' of gitlab.com:minds/front into feat/notification-e2e-1640

Compare with previous version

closed