feat(near-api-js): wallet sign-in page redirection is now optional

[DevSlashRichie]

Motivation

Sometimes, we want the sign-in process not to redirect the whole page, but to be open in a popup window.
As mentioned in #1010

Description

She added to different methods (requestSignInLink, requestSignInLink, and completeSignInWithAccessKeyLink) to the WalletConnection, to get the needed URLs to the NEAR wallet in order to manually process them like in a popup or _blank redirection.

Checklist

• Read the contributing guidelines
• Commit messages follow the conventional commits spec
• Performed a self-review of the PR
• Added automated tests (The methods added are already tested as they are executed by the ones with a written test)
• Manually tested the change

[changeset-bot]

⚠️ No Changeset found

Latest commit: 30f6fa1

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[frol]

@DevSlashRichie We are bringing near-api-js back to be actively maintained. This change makes total sense to me, but it needs to be rebased now (probably faster to just manually re-implement it as a lot of files have actually moved to other places). Do you have the capacity to rebase?

[denbite]

the idea of having methods for generating a Wallet URL without redirecting looks also reasonable to me and was implemented in #1260

I'd suggest closing this one so we don't leave it out of date

[vikinatora]

Resolved by #1260.