fix(@aws-amplify/analytics): avoid return statement in the beforeunload function

[powerful23]

Issue #, if available:
fixes #2088
In the compiled js file, there will be a return statement if the function is async which will cause the issue. By eliminating the async should fix it.
Description of changes:

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

[manueliglesias]

Doing async stuff inside the beforeunload is not guaranteed to complete/succeed.

amplify-js/packages/analytics/src/trackers/SessionTracker.ts

Line 162 in ebb9c9e

window.addEventListener("beforeunload", this._trackBeforeUnload, false);

We need to explore better approaches for the future (e.g. service workers)

Approving as to help people using IE11

[powerful23]

@manueliglesias Agree.

[github-actions]

This pull request has been automatically locked since there hasn't been any recent activity after it was closed. Please open a new issue for related bugs.

Looking for a help forum? We recommend joining the Amplify Community Discord server *-help channels or Discussions for those types of questions.