[Manual Mapping] Maintain the mapping for more than 1 URL in the localstorage.

[atopper]

Description

The mapping for 1 URL is deleted if the URL is changed, and a new mapping is started. Work can be lost easily this way. This change maintains multiple URls in the local storage

Eventually, perhaps, we should set a TTL of URL mappings. They could grow over time.

Related Issue

Motivation and Context

Ensure mappings are saved.

How Has This Been Tested?

Manually and ran tests.

Screenshots (if appropriate):

Types of changes

I'll say bug, although 'new feature' might apply.

• [x] Bug fix (non-breaking change which fixes an issue)
• [ ] New feature (non-breaking change which adds functionality)
• [ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• [x] I have signed the Adobe Open Source CLA.
• [x] My code follows the code style of this project.
• [ ] My change requires a change to the documentation.
• [ ] I have updated the documentation accordingly.
• [x] I have read the CONTRIBUTING document.
• [ ] I have added tests to cover my changes.
• [x] All new and existing tests passed.

[atopper]

@catalan-adobe @arumsey I can't seem to add reviewers (last of permission I assume), so hopefully this will bring it to your attention.

[atopper]

Do you think instead of the URL as the 'key', to use the hostname instead, so that any subpages are saved in the same mapping? Probably not needed for the pre-sales demo, but it is something that I keep wondering about.