Skip to content
This repository has been archived by the owner on Jan 9, 2024. It is now read-only.

Update FathomAnalytics.php #4

Merged
merged 2 commits into from
Jul 12, 2021
Merged

Conversation

austinginder
Copy link
Contributor

Fix for Fathom v3 launch to resolve issues #3

Fix for Fathom v3 launch
Copy link

@tectonic-pg tectonic-pg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the submit button for 2FA needs to be updated as well

$this->browser->submitForm('Login', [
    'one_time_password' => $oneTimePassword,
]);

@austinginder
Copy link
Contributor Author

Updated

@ndeblauw
Copy link

ndeblauw commented Jul 5, 2021

I confirm, this fix resolves the issue. Nice work @austinginder

@ndeblauw
Copy link

@mpociot @sschlein Could you please merge this (small, innocent & bug fixing) PR into the master branch and release it? Shouldn't take more than 5 minutes, and it really helps us forward!

@sschlein sschlein merged commit 3da1a3e into beyondcode:master Jul 12, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants