Opened on 11/24/2015 at 03:42:32 PM
Closed on 11/26/2015 at 09:30:47 AM
Last modified on 11/26/2015 at 11:01:17 AM
#3346 closed change (fixed)
Remove edge subscription form
Reported by: | saroyanm | Assignee: | saroyanm |
---|---|---|---|
Priority: | P1 | Milestone: | |
Module: | Websites | Keywords: | |
Cc: | fhd, oleksandr, matze, lisabielik | Blocked By: | |
Blocking: | Platform: | Unknown / Cross platform | |
Ready: | yes | Confidential: | no |
Tester: | Unknown | Verified working: | no |
Review URL(s): |
Description (last modified by saroyanm)
Background
Subscription confirmation mails are blocked by various providers (mostly Microsoft) as described by #3204, this may require a more sophisticated mail solution in general on our side which we do not have right now, so while our users can't get emails doesn't make sense to have this form.
What to change
Remove the subscription form implementation from https://adblockplus.org/en/edge and revert back as soon the issue is fixed.
Add text below instead of form:
Please check back later for a version of Adblock Plus for Microsoft Edge.
Attachments (0)
Change History (10)
comment:1 Changed on 11/24/2015 at 04:10:22 PM by saroyanm
- Owner set to saroyanm
comment:2 Changed on 11/24/2015 at 04:39:59 PM by saroyanm
- Review URL(s) modified (diff)
- Status changed from new to reviewing
comment:5 in reply to: ↑ 4 ; follow-up: ↓ 6 Changed on 11/25/2015 at 02:03:23 PM by lisabielik
Replying to saroyanm:
@lisa we would like to add a message to desktop and mobile version of the page saying something like Please check back later for a version of Adblock Plus for Microsoft Edge. can you please review the text ?
The problem is that on mobile version we are not showing the "Adblock Plus for Microsoft Edge" promo image and it's no message visible to mobile users describing what the page is for.
The text will appear instead of Install button.
If you are okey with text I'll update the ticket and add the text.
I think the current text is fine: Please check back later for a version of Adblock Plus for Microsoft Edge.
comment:6 in reply to: ↑ 5 Changed on 11/25/2015 at 05:03:09 PM by saroyanm
- Description modified (diff)
Replying to lisabielik:
I think the current text is fine: Please check back later for a version of Adblock Plus for Microsoft Edge.
Thanks
comment:7 Changed on 11/26/2015 at 09:30:47 AM by saroyanm
- Resolution set to fixed
- Status changed from reviewing to closed
comment:8 follow-ups: ↓ 9 ↓ 10 Changed on 11/26/2015 at 10:32:40 AM by oleksandr
I have just realized that this also shows the Microsoft Edge message to ALL users on mobile. Its great for Microsoft Edge, of course :), but I think it should show the message only for those who navigate to the page using Microsoft Edge on Windows 10 Mobile.
comment:9 in reply to: ↑ 8 Changed on 11/26/2015 at 10:51:32 AM by saroyanm
Replying to oleksandr:
I have just realized that this also shows the Microsoft Edge message to ALL users on mobile. Its great for Microsoft Edge, of course :), but I think it should show the message only for those who navigate to the page using Microsoft Edge on Windows 10 Mobile.
I think it's shows the message to on all pages, hmm.. I think it's should be available only on edge page.. praparing patch.
comment:10 in reply to: ↑ 8 Changed on 11/26/2015 at 11:01:17 AM by saroyanm
Replying to oleksandr:
I have just realized that this also shows the Microsoft Edge message to ALL users on mobile. Its great for Microsoft Edge, of course :), but I think it should show the message only for those who navigate to the page using Microsoft Edge on Windows 10 Mobile.
YOu are right, the patch is ready.
@lisa we would like to add a message to desktop and mobile version of the page saying something like Please check back later for a version of Adblock Plus for Microsoft Edge. can you please review the text ?
The problem is that on mobile version we are not showing the "Adblock Plus for Microsoft Edge" promo image and it's no message visible to mobile users describing what the page is for.
The text will appear instead of Install button.
If you are okey with text I'll update the ticket and add the text.