1679854 - Update error message for add-ons try to use storage.sync with a temporary ID

Status: RESOLVED FIXED

(WebExtensions :: General, enhancement, P3)

Description

[Caitlin Neiman [:caitmuenster]]

Based on this thread: https://news.ycombinator.com/item?id=25218240

Currently, if an extension with a temporary ID tries to use storage.sync, they see this error message:

'Error: The storage API will not work with a temporary addon ID. Please add an explicit addon ID to your manifest. For more information see https://bugzil.la/1323228.'

The error message should be updated to:

'Error: The storage API will not work with a temporary addon ID. Please add an explicit addon ID to your manifest. For more information see https://bit.ly/3lntYMe.'

Comment 1

[Luca Greco [:rpl] [:luca] [:lgreco]]

(In reply to Caitlin Neiman [:caitmuenster] from comment #0)

'Error: The storage API will not work with a temporary addon ID. Please add an explicit addon ID to your manifest. For more information see https://bit.ly/3lntYMe.'

Hi Caitlin,
Thanks for filing this bug!

It would be great if we could try to ask for the https://extensionworkshop.com domain to be added to the domains allowed by the https://mzl.la/ shorten url prefix, from a quick look in bugzilla it seems that we can ask that by filing an issue in the "Marketing::Social Media" bugzilla component, e.g. see Bug 668191 and Bug 885722.

Would you mind to file that bug and link it as a blocker for this one?

Comment 2

[Caitlin Neiman [:caitmuenster]]

Hey Luca! I talked to Liz Hull and we worked this out directly with our Bitly rep.

The new shortened link for the relevant URL is https://mzl.la/3lPk1aE. :)

Comment 3

[Luca Greco [:rpl] [:luca] [:lgreco]]

Attachment: Bug 1679854 - Updated link in storage.sync temporary addon id error message to point it to an extensionworkshop doc page. r?mixedpuppy!

Comment 4

[Luca Greco [:rpl] [:luca] [:lgreco]]

Thanks Caitlin, the actual change to be applied are pretty small (and it wouldn't be that much interesting even as a good first bug) and so I just created the "two lines" patch and attached it to this issue.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/599cc7b5991b
Updated link in storage.sync temporary addon id error message to point it to an extensionworkshop doc page. r=mixedpuppy

Comment 6

[Narcis Beleuzu [:NarcisB]]

https://hg.mozilla.org/mozilla-central/rev/599cc7b5991b