Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#2335 closed change (fixed)

Create landing page for successful email verification

Reported by: saroyanm Assignee: saroyanm
Priority: P2 Milestone:
Module: Websites Keywords:
Cc: sebastian, sven, annlee@…, lisabielik Blocked By:
Blocking: #2213, #2234, #2436 Platform: Unknown
Ready: yes Confidential: no
Tester: Verified working: no
Review URL(s):

http://codereview.adblockplus.org/6216724063977472/
http://codereview.adblockplus.org/5316826510655488/
http://codereview.adblockplus.org/5686635006001152/

Description (last modified by sebastian)

Background

In #2213 we are creating landing page for Adblock Browser for Android beta launch, but we also should have subscription field on that page where users can subscribe for being notified about Adblock Plus IOS browser launch. Users will get confirmation email (#2234) after subscription. So we need also a landing page where we can redirect users to that have verified their email address.

What to change

Add successful verification page:
Url -> ​https://adblockplus.org/en/adblock-browser/verification-success
Text -> Thanks for your submission! We'll email you before the launch.

Add meta tag bellow to the head section of both pages:
<meta name="robots" content="noindex">

Change History (18)

comment:1 Changed 5 years ago by saroyanm

  • Description modified (diff)

comment:2 Changed 5 years ago by saroyanm

  • Description modified (diff)
  • Priority changed from Unknown to P2
  • Ready set

comment:3 Changed 5 years ago by saroyanm

  • Cc sven added

@Sven please let me know if you would like to have some special design of showing that notification, otherwise I'll go with default heading.

comment:4 Changed 5 years ago by saroyanm

  • Description modified (diff)

comment:5 follow-up: Changed 5 years ago by sebastian

  • Cc annlee@… lisabielik added; annlee@… removed

The texts are semantically correct, but I suppose Ann-Lee or Lisa could still improve the phrasing.

comment:6 in reply to: ↑ 5 Changed 5 years ago by saroyanm

Replying to sebastian:

The texts are semantically correct, but I suppose Ann-Lee or Lisa could still improve the phrasing.

Please note that text is already improved by Lisa. But, sure if there will be more improvements then we can go with new text. Anyway current text looks to be good enough to give a go I think.

comment:7 Changed 5 years ago by sebastian

Ah ok, didn't realize. Then it should be fine.

comment:8 Changed 5 years ago by sebastian

  • Description modified (diff)
  • Summary changed from Two verification landing pages on adblockplus.org to Create landing page for successful email verification

I've updated the issue, since as discussed with saroyanm and specified in #2234, there will only be a one landing page for successful email verification.

comment:9 Changed 5 years ago by sebastian

  • Blocking 2234 added

comment:10 Changed 5 years ago by saroyanm

  • Owner set to saroyanm
  • Review URL(s) modified (diff)

comment:11 Changed 5 years ago by saroyanm

  • Status changed from new to reviewing

comment:12 Changed 5 years ago by saroyanm

I'll close the issue when the actually page will be created, I do not want to create it now, to be 100% free from leak while the path that mentioned in the ticket is sub directory of adblock-browser.

comment:13 Changed 5 years ago by saroyanm

  • Blocking 2436 added

comment:14 follow-up: Changed 5 years ago by Ross

I still need an exported .xml of this page or access to the patch/change on Google code to test this one.

comment:15 in reply to: ↑ 14 Changed 5 years ago by saroyanm

Replying to Ross:

I still need an exported .xml of this page or access to the patch/change on Google code to test this one.

Please check initial mail with the attachment, the XML should be available there.

comment:16 Changed 5 years ago by saroyanm

  • Review URL(s) modified (diff)

comment:17 Changed 5 years ago by saroyanm

  • Resolution set to fixed
  • Review URL(s) modified (diff)
  • Status changed from reviewing to closed

comment:18 Changed 5 years ago by saroyanm

  • Sensitive unset
Note: See TracTickets for help on using tickets.