-
Notifications
You must be signed in to change notification settings - Fork 17
Conversation
I think it looks fine, but Chuck will be a better reviewer than me for this code so flagging him |
@javaun , @chuckharmston : In light of the urgency of this change request from Mika, I would ask that you please let me know by the end of the day tomorrow if at all possible (Wednesday, 3/20); otherwise my plan will be to:
|
My thoughts:
Thanks for handling this, Bianca! |
Filed mozilla/testpilot#4045 to make sure the Test Pilot site gets updated. @clouserw, can you make sure somebody is able to work on that? |
Thanks @chuckharmston for the quick feedback! Just one clarification for you:
Just to confirm, since the privacy policy page's URL is based on the extension's AMO page URL, you would rather we update the AMO page URL from https://addons.mozilla.org/en-US/firefox/addon/price-wise/ to https://addons.mozilla.org/en-US/firefox/addon/price-tracker/ ? Per jorgev, it is not possible to redirect from the former to the latter. |
Yeah, that seems like the sort of thing that is necessary for us to do. Since this is effectively a dormant project, I'm not too concerned about the consequences of the URL change, especially since there's still a link on Test Pilot's site that can be changed. |
Find and replace all instances of "Price Wise", "price-wise" and "price_wise" (case insensitive) with the following exceptions: * The telemetry event category, 'extension.price_wise' for continuity. * The survey URLs (for [feedback](https://qsurvey.mozilla.com/s3/price-wise) linked to in the popup and the now flagged off [UR study](https://qsurvey.mozilla.com/s3/Price-Wise-Research-Study)). Note: Updating the name of the project in `package.json` and calling `npm install` also triggered adding an additional [`optional`](https://docs.npmjs.com/files/package-lock.json#optional) key in `package-lock.json` for some devDeps.
Got an AMO pre-review approval over e-mail from @wagnerand , so going to merge. Also: thanks rhelmer! |
Any ETA on when the add-on will be published at https://addons.mozilla.org/en-US/firefox/addon/price-tracker/? |
@pdehaan it's been submitted to AMO for review. Fingers crossed... |
It was published about an hour ago. |
This should be in production now, per mozilla/testpilot#4045 (comment) |
@javaun , @chuckharmston Requesting your approval on the following noteworthy items:
'extension.price_wise'
for continuity.Find and replace all instances of "Price Wise", "price-wise" and "price_wise" (case insensitive) with the following exceptions:
Note: Updating the name of the project in
package.json
and callingnpm install
also triggered adding an additionaloptional
key inpackage-lock.json
for some devDeps.