[#37] Update design of confirmation letter #117

cleac · 2016-09-17T07:40:27Z

Resolves #37

Implement the updated design of confirmation letter.

@webknjaz @bluebirrrrd @anxolerd: give your suggestions of text improvements

webknjaz · 2016-09-17T21:00:02Z

src/GDGUkraine/templates/email/confirmation.html

+        <meta http-equiv="Content-Type" content="text/html; charset=UTF-8"/>
+        <meta name="viewport" content="width=device-width, initial-scale=1.0"/>
+    </head>
+    <body style="marin: 8px; padding: 0px; max-width: 768px">


webknjaz · 2016-09-17T21:01:55Z

src/GDGUkraine/templates/email/confirmation.html

+        <center><img style="width: 200px" width="200px" src="/img/gdg_ukraine_logo.png"></center>
+        <p style="margin-left: 20px; margin-bottom: 20px; font-family: Lato, Arial, sans-serif">Hey, pal!</p>
+        <p style="margin-left: 20px; font-family: Lato, Arial, sans-serif">
+        We are writing to you, because you have registered to "{{ event.title }}". We just want to clarify, whether you are going to attend the event, or you have another plans. You can get more info about an event right <a href="{{}}">here</a>.


+80 chars lines is more readable

not sure about comma before "whether"

https://www.usingenglish.com/forum/threads/52133-writing-you-vs-writing-to-you

webknjaz · 2016-09-17T21:02:58Z

src/GDGUkraine/templates/email/confirmation.html

+        <p style="margin-left: 20px; font-family: Lato, Arial, sans-serif">
+        We are writing to you, because you have registered to "{{ event.title }}". We just want to clarify, whether you are going to attend the event, or you have another plans. You can get more info about an event right <a href="{{}}">here</a>.
+        </p>
+        <p style="margin: 0px 20px; font-family: Lato, Arial, sans-serif">If you are going to attend, press the button below.</p>


Again, I don't want to scroll horizontally

webknjaz · 2016-09-17T21:03:23Z

src/GDGUkraine/templates/email/confirmation.html

+        We are writing to you, because you have registered to "{{ event.title }}". We just want to clarify, whether you are going to attend the event, or you have another plans. You can get more info about an event right <a href="{{}}">here</a>.
+        </p>
+        <p style="margin: 0px 20px; font-family: Lato, Arial, sans-serif">If you are going to attend, press the button below.</p>
+        <p style="margin: 0px 20px; font-family: Lato, Arial, sans-serif">If no, just ignore this email.</p>


Otherwise?

+1 to Otherwise. Seems better for me

both are valid

webknjaz · 2016-09-17T21:03:53Z

src/GDGUkraine/templates/email/confirmation.html

+        <p style="margin: 0px 20px; font-family: Lato, Arial, sans-serif">If no, just ignore this email.</p>
+        <a href="{{ confirm.url | e }}" style="text-decoration: none">
+            <div style="margin: 10px auto; background-color: blue; width: 160px; height: 35px; color: white; text-align: center; vertical-align: baseline; border-radius: 3px; border-color: blue" width="160px" height="35px" bgcolor="blue">
+                <div style="padding: 6px; font-family: Lato, Arial, sans-serif">I am going to attend</div>


attend -> come

Make the button text shorter. Smth like I'll be there or I will come...

webknjaz · 2016-09-17T21:04:34Z

src/GDGUkraine/templates/email/confirmation.html

+        <center><img style="width: 200px" width="200px" src="/img/gdg_ukraine_logo.png"></center>
+        <p style="margin-left: 20px; margin-bottom: 20px; font-family: Lato, Arial, sans-serif">Hey, pal!</p>
+        <p style="margin-left: 20px; font-family: Lato, Arial, sans-serif">
+        We are writing to you, because you have registered to "{{ event.title }}". We just want to clarify, whether you are going to attend the event, or you have another plans. You can get more info about an event right <a href="{{}}">here</a>.


not sure about comma before "whether"

webknjaz

The text may still need some fixes

webknjaz · 2016-09-17T21:06:28Z

src/GDGUkraine/templates/email/confirmation.html

+    </head>
+    <body style="marin: 8px; padding: 0px; max-width: 768px">
+        <center><img style="width: 200px" width="200px" src="/img/gdg_ukraine_logo.png"></center>
+        <p style="margin-left: 20px; margin-bottom: 20px; font-family: Lato, Arial, sans-serif">Hey, pal!</p>


pal -> rand('dude', 'man', 'pal', etc)

I am thinking about using participant's name here:
Hi {{ u.name or 'friend' }}

As for pal, dude, man and so on, I believe we should replace it with something less gender-specific. You know, there can be female participants as well... =)

@anxolerd pal is not gender-specific, as I think.... But that's good idea about {{ u.name or 'placeholder' }}

anxolerd · 2016-09-17T21:17:54Z

src/GDGUkraine/templates/email/confirmation.html

+        We are writing to you, because you have registered to "{{ event.title }}". We just want to clarify, whether you are going to attend the event, or you have another plans. You can get more info about an event right <a href="{{}}">here</a>.
+        </p>
+        <p style="margin: 0px 20px; font-family: Lato, Arial, sans-serif">If you are going to attend, press the button below.</p>
+        <p style="margin: 0px 20px; font-family: Lato, Arial, sans-serif">If no, just ignore this email.</p>


Otherwise?

anxolerd · 2016-09-17T21:24:45Z

src/GDGUkraine/templates/email/confirmation.html

+    </head>
+    <body style="marin: 8px; padding: 0px; max-width: 768px">
+        <center><img style="width: 200px" width="200px" src="/img/gdg_ukraine_logo.png"></center>
+        <p style="margin-left: 20px; margin-bottom: 20px; font-family: Lato, Arial, sans-serif">Hey, pal!</p>


I am thinking about using participant's name here:
Hi {{ u.name or 'friend' }}

As for pal, dude, man and so on, I believe we should replace it with something less gender-specific. You know, there can be female participants as well... =)

anxolerd · 2016-09-17T21:26:51Z

src/GDGUkraine/templates/email/confirmation.html

+        <p style="margin: 0px 20px; font-family: Lato, Arial, sans-serif">If no, just ignore this email.</p>
+        <a href="{{ confirm.url | e }}" style="text-decoration: none">
+            <div style="margin: 10px auto; background-color: blue; width: 160px; height: 35px; color: white; text-align: center; vertical-align: baseline; border-radius: 3px; border-color: blue" width="160px" height="35px" bgcolor="blue">
+                <div style="padding: 6px; font-family: Lato, Arial, sans-serif">I am going to attend</div>


Make the button text shorter. Smth like I'll be there or I will come...

anxolerd · 2016-09-17T21:28:52Z

src/GDGUkraine/templates/email/confirmation.html

+        </a>
+        <p style="margin-left: 20px; font-family: Lato, Arial, sans-serif">
+        Sincerely yours,<br>
+        friends from GDG Ukraine


We have a group related to GDG event. Probably we should be more specific and use it.

Truth. Will fix it

anxolerd · 2016-09-17T21:35:24Z

src/GDGUkraine/templates/email/confirmation.html

+        <p style="margin: 0px 20px; font-family: Lato, Arial, sans-serif">If you are going to attend, press the button below.</p>
+        <p style="margin: 0px 20px; font-family: Lato, Arial, sans-serif">If no, just ignore this email.</p>
+        <a href="{{ confirm.url | e }}" style="text-decoration: none">
+            <div style="margin: 10px auto; background-color: blue; width: 160px; height: 35px; color: white; text-align: center; vertical-align: baseline; border-radius: 3px; border-color: blue" width="160px" height="35px" bgcolor="blue">


What about not hardcoding height?

(thunderbird)

Thx for checking. I think I'll fix it with smaller text in button

Why not to use rem on text nodes, and em on thinks, that attached to nodes? This will bring more responsibility to letter.

Will this work for the most of email clients?

@webknjaz, @ValkoVolodya I was not sure it does, so I used px. Most guides used it, so I did so 😄

anxolerd · 2016-09-17T21:38:04Z

src/GDGUkraine/templates/email/confirmation.html

+        <meta name="viewport" content="width=device-width, initial-scale=1.0"/>
+    </head>
+    <body style="marin: 8px; padding: 0px; max-width: 768px">
+        <center><img style="width: 200px" width="200px" src="/img/gdg_ukraine_logo.png"></center>


src="/img/..." This is bad. Make the url absolute (starting from https)

By the way, you can use group-specific logo here.

There is <base href="https://gdg.org.ua"/> in the headings of letter, so the absolute url is not required.

And what about group-specific logo - will think about it

anxolerd · 2016-09-17T21:49:30Z

src/GDGUkraine/templates/email/confirmation.html

+    <body style="marin: 8px; padding: 0px; max-width: 768px">
+        <center><img style="width: 200px" width="200px" src="/img/gdg_ukraine_logo.png"></center>
+        <p style="margin-left: 20px; margin-bottom: 20px; font-family: Lato, Arial, sans-serif">Hey, pal!</p>
+        <p style="margin-left: 20px; font-family: Lato, Arial, sans-serif">


Is it really impossible to write paragraph styles only once?

Only in case of using some tool, which would inline everything from CSS declarations into html tags. It's a must for web UI based email clients to have styles inline.

@webknjaz: oh crap :(

anxolerd · 2016-09-17T21:55:35Z

src/GDGUkraine/templates/email/confirmation.html

+        <center><img style="width: 200px" width="200px" src="/img/gdg_ukraine_logo.png"></center>
+        <p style="margin-left: 20px; margin-bottom: 20px; font-family: Lato, Arial, sans-serif">Hey, pal!</p>
+        <p style="margin-left: 20px; font-family: Lato, Arial, sans-serif">
+        We are writing to you, because you have registered to "{{ event.title }}". We just want to clarify, whether you are going to attend the event, or you have another plans. You can get more info about an event right <a href="{{}}">here</a>.


As for this paragraph, user sees no value to himself (and no reason in consequence) to click the button. What about smth like:

We are glad you are interested in {{ event.title }}. There is one small step left to complete your registration...

?

We have to give some value to user so he/she could see the reason to click that button, not just ignore this email.

I don't like how it spells. It's a bit... business-formal or somehow like that.

Also that staff about one little step, tells that user has to do more stuff for us (while noone likes to do a lot of staff for someone - event just click on a button). And, also, it generates more text he has to read. Do you read all those long emails, the only purpose of which is to confirm that you are actually going to go somewhere?

I think, it's better to make it like that:

begin-part

Thanks for registering to the event "{{ event.title }}".
We are really looking forward to see you there, but we must clarify that you will come. If you will, press "I will come".

If not, no action is needed, just ignore this letter.

end-part

This is kinda draft and has some issues.

http://www.english-for-students.com/What-is-the-difference-between-Staff-and-Stuff.html

@webknjaz thanks :-) Didn't knew that XD

knew -> know

anxolerd · 2016-09-18T13:05:05Z

Hey, @bluebirrrrd ! We know you're a super-dupa busy person, but could you please help us with the text of confirmation letter? We really need your help and advise ;)

bluebirrrrd · 2016-09-19T08:06:36Z

Hey guys,

I'll skip the part with criticism and just modify the @cleac's version below.

begin-part

Thanks for registering to the event "{{ event.title }}".
Please confirm your attendance by clicking the button below.

<div style="text-align:center">
  <a href="http://reg.gdg.org.ua/some/confirmation/url" style="display: block; background-color: #1c90f3;
    width: 100px; color: #eee; border-radius: 5px; text-decoration: none; padding: 10px; font-family: arial;">I will come</a>
</div>

If you have decided not to come, just ignore this letter.

end-part

Also, adding some time limits for confirmation would be nice. I skipped this part since we don't have any features for that at the moment.

[Advice] Please read The art of business correspondence by Sasha Karepina before writing any letters of confirmation. I believe all of you will find it very helpful for further mailing (& discussions on github as well).

webknjaz · 2016-11-16T12:22:17Z

@cleac

cleac · 2016-11-18T06:48:36Z

Currently have some work and univ's routine. Will do ASAP 😄

webknjaz · 2016-12-21T21:48:45Z

@cleac status?

cleac · 2016-12-22T04:43:34Z

@webknjaz WIP cause too much time left from time I started this PR

anxolerd · 2017-01-21T16:14:41Z

@cleac: ping
What is left to be done? Maybe @webknjaz and I can accomplish that while you are busy?

anxolerd · 2017-02-12T23:10:36Z

@cleac ping

cleac · 2017-03-05T08:53:19Z

Hi all, sorry for losing touch. I will finish this PR today, currently working on it 😃

cleac · 2017-03-05T11:16:51Z

@webknjaz @anxolerd

I decided to use mjml framework for this. After trying several of them, it looks the best and easiest as for me.

I will make detached header with GDG logo and text which have to be in every single email, like hi, pal in separate mjml file, like header.mjml for it to be easily included into every email template file and mask all html files in that folder in gitignore. Here is a conceptual implementation of concept: example. I'll make implementation in repo and push it here a bit later today

The only drawback I've found - we will have to put only absolute urls in the letter.

webknjaz · 2017-03-05T12:37:03Z

The only drawback I've found - we will have to put only absolute urls in the letter.

sounds ok

webknjaz · 2017-03-05T12:38:45Z

@cleac I've implemented a set of url_for() resolvers, which produce absolute URLs, they are available in jinja2 templates

cleac · 2017-03-05T12:41:06Z

I've implemented a set of url_for() resolvers, which produce absolute URLs, they are available in jinja2 templates

@webknjaz awesome! :-)

cleac · 2017-03-05T15:40:33Z

@webknjaz is there any doc on url_for you mentioned above? I'd like to use it, but don't know where should I search it :-)

webknjaz · 2017-03-05T16:16:32Z

https://github.com/GDG-Ukraine/gdg.org.ua/blob/master/src/tests/test_utils.py#L16-L76
https://github.com/GDG-Ukraine/gdg.org.ua/blob/master/src/GDGUkraine/lib/utils/url.py#L201-L295

cleac · 2017-03-05T16:45:49Z

It seems that thats all on task :-)
@webknjaz @anxolerd @bluebirrrrd please, check it out

webknjaz · 2017-03-05T21:31:10Z

src/GDGUkraine/templates/email/_gdg_logo.mjml

@@ -0,0 +1,6 @@
+<mj-section>
+  <mj-column>
+      <mj-image width="200" src="/img/gdg_ukraine_logo.png"></mj-image>


Any chance to have templated src value via url_for_static?

Haven't tried it, will look through

webknjaz · 2017-03-06T08:06:53Z

Makefile

@@ -19,6 +19,7 @@ TOX=$(PENV_BIN_PATH)tox
 USER_SHELL=/bin/zsh
 ACTIVATE_ENV=if test -d ./$(PENV); then . ./$(PENV)/bin/activate; fi; if test -f ./.exports; then . ./.exports; fi
 USE_NVM=if test -d ~/.nvm; then . ~/.nvm/nvm.sh; nvm use 5.0.0 ; fi
+EMAIL_DIRECTORY=src/GDGUkraine/templates/email


We should make make learn about dotenv ^^

webknjaz · 2017-03-06T08:07:36Z

Makefile

@@ -144,3 +145,6 @@ stop-prod:
 .PHONY: environ-regen
 environ-regen:
 	bin/mk-environ-file.sh
+
+mjml:
+	@bin/build_mjml.sh ${EMAIL_DIRECTORY}


I think you shouldn't suppress output here

webknjaz · 2017-03-06T08:09:03Z

Makefile

@@ -144,3 +145,6 @@ stop-prod:
 .PHONY: environ-regen
 environ-regen:
 	bin/mk-environ-file.sh
+
+mjml:


You may also want to add front-deps target as a dependency and extend it to install utility you need :)

cleac · 2017-03-06T15:32:19Z

I'll write down some tests later today

webknjaz · 2017-03-06T19:00:35Z

Makefile

@@ -142,9 +143,10 @@ stop-prod:

 .env: environ-regen

-.PHONY: environ-regen
+.PHONY: environ-regen mjml


why do you want to regen mjml along with .env?

@webknjaz : emmm... Is someone going to call make .PHONY? 'R 'u mad?

oh, and this is an incorrect syntax for .phony

webknjaz · 2017-03-06T19:28:41Z

bin/build_mjml.sh

@@ -18,11 +18,11 @@ END
 MJML_REGEX="^[^_].+\.mjml$"

 if [ "$1" = 'help' -o "$1" = 'h' -o "$1" = '--help' ]; then
-  echo "$USAGE"
+  echo -n "$USAGE\n"


This is a nonsense. Just don't use -n here.

WTF?
Do I understand that correctly that you disable default trailing newline and print yours instead? Why would you ever need that?

My bad, I just did :%s/printf/echo -n/g 😅 😅 😅 😅

anxolerd · 2017-03-06T20:57:37Z

Makefile

@@ -13,12 +13,14 @@ OPEN_URL=xdg-open
 MIGRATOR=$(PENV_BIN_PATH)alembic -c config/alembic.ini
 BWR=bower
 NPM=npm
+MJML=mjml


By the way I literally have no idea why have both you and @webknjaz created these aliases.
Can anyone explain that to me, please?

🤷‍♂️

@anxolerd the same reason, why there are predefined $(CC) and $(CXX) aliases. For easier possibility to override them, if they won't be in PATH

cleac · 2017-03-13T08:09:10Z

@webknjaz @anxolerd I won't have much time this week (preparing for a new position interview), please check it out and leave comments for reasonable issues. I'll add several unit tests a bit later in another PR if you don't mind 😉

webknjaz · 2017-03-14T06:54:40Z

bin/build_mjml.sh

@@ -18,11 +18,11 @@ END
 MJML_REGEX="^[^_].+\.mjml$"

 if [ "$1" = 'help' -o "$1" = 'h' -o "$1" = '--help' ]; then
-  echo -n "$USAGE\n"


@webknjaz Why do I need this USAGE? Just for little help if someone will want to use this script in other place without reading the code. I wasn't going to add this script to the Makefile while writing it, so I decided to add some help and now I have no idea why I should delete it 🤷‍♂️

no, why did you remove -n? (again)

anxolerd · 2017-03-14T23:11:20Z

bin/build_mjml.sh

+  else
+    if [ -z $dirs ]
+    then
+      dirs=$arg


what if arg contains spaces?

anxolerd · 2017-03-14T23:13:56Z

bin/build_mjml.sh

+  if [ "$arg" = '-m' -o "$arg" = '--minify' ]; then
+    minify='-m'
+  else
+    if [ -z $dirs ]


if dirs contains spaces this fails with error

anxolerd · 2017-03-14T23:14:07Z

bin/build_mjml.sh

+  fi
+done
+
+if [ -z $dirs ]; then 


if dirs contains spaces this fails with error

anxolerd · 2017-03-14T23:15:07Z

bin/build_mjml.sh

+done
+
+if [ -z $dirs ]; then 
+  echo -n 'Please, specify directory, where the files to build are!\n'


echo 'Please, specify ...to build are!'

BTW, what abound sending this to stderr ?

anxolerd · 2017-03-14T23:16:52Z

bin/build_mjml.sh

+fi
+
+for dirname in $dirs; do
+  if [ -d $dirname ]; then


Fails if dirname contains spaces

anxolerd · 2017-03-14T23:18:29Z

bin/build_mjml.sh

+for dirname in $dirs; do
+  if [ -d $dirname ]; then
+    echo -n "Building for directory $dirname "
+    for file in $(ls $dirname | grep -E $MJML_REGEX )


probably $dirname and $MJML_REGEX should be quoted

anxolerd · 2017-03-14T23:19:06Z

bin/build_mjml.sh

+      if [ $? -eq 0 ]; then
+        echo -n '.'
+      else
+        echo -n 'ERROR\nAn error occured, please check it out\n'


what about sending this to stderr ?

anxolerd · 2017-03-14T23:19:20Z

bin/build_mjml.sh

+        exit 3
+      fi
+    done
+    echo -n ' Done\n'


echo ' Done'

anxolerd · 2017-03-14T23:19:46Z

bin/build_mjml.sh

+    done
+    echo -n ' Done\n'
+  else 
+    echo -n "\nDirectory '$dirname' was not found: skipping\n"


echo "\nDirectory ... skipping"

cleac · 2017-04-26T14:31:03Z

Finish #117

How to build mjml emails: 1. Install mjml globally from npm repository 2. Run make mjml everytime you edit email All the emails templates now have to be in directory src/GDGUkraine/templates/email. If not, add directory to EMAIL_DIRECTORY in Makefile - Add mjml tempaltes building - Add _gdg_logo.mjml file with gdg logo in it

- Add mjml_build script to .PHONY and frontend-deps targets - Make gdg_logo take absolute url - Make build_mjml.sh use `echo -n` instead of `printf` - Remove build html file from diff

- Fix usage of .PHONY target - Fix `echo -n` issue

- Make errors be thrown to stderr

cleac · 2017-05-29T05:49:32Z

@webknjaz @anxolerd Guys, can you pls check out this PR? I've fixed the script

cleac added the Stage: In Progress label Sep 17, 2016

webknjaz reviewed Sep 17, 2016

View reviewed changes

webknjaz suggested changes Sep 17, 2016

View reviewed changes

anxolerd reviewed Sep 17, 2016

View reviewed changes

webknjaz assigned webknjaz, anxolerd, bluebirrrrd and cleac Sep 18, 2016

cleac force-pushed the master branch 2 times, most recently from 74621e8 to bed6b74 Compare December 21, 2016 16:00

cleac force-pushed the master branch from bed6b74 to 98b05bf Compare March 5, 2017 08:53

webknjaz reviewed Mar 5, 2017

View reviewed changes

webknjaz reviewed Mar 6, 2017

View reviewed changes

anxolerd reviewed Mar 6, 2017

View reviewed changes

webknjaz reviewed Mar 14, 2017

View reviewed changes

anxolerd reviewed Mar 14, 2017

View reviewed changes

bin/build_mjml.sh Outdated

else

if [ -z $dirs ]

then

dirs=$arg

Copy link

Contributor

anxolerd Mar 14, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what if arg contains spaces?

anxolerd reviewed Mar 14, 2017

View reviewed changes

bin/build_mjml.sh Outdated

fi

done

if [ -z $dirs ]; then

Copy link

Contributor

anxolerd Mar 14, 2017 •

edited

Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if dirs contains spaces this fails with error

anxolerd reviewed Mar 14, 2017

View reviewed changes

bin/build_mjml.sh Outdated

fi

for dirname in $dirs; do

if [ -d $dirname ]; then

Copy link

Contributor

anxolerd Mar 14, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fails if dirname contains spaces

anxolerd reviewed Mar 14, 2017

View reviewed changes

bin/build_mjml.sh Outdated

exit 3

fi

done

echo -n ' Done\n'

Copy link

Contributor

anxolerd Mar 14, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

echo ' Done'

anxolerd reviewed Mar 14, 2017

View reviewed changes

cleac and others added 6 commits May 29, 2017 08:19

Implementat of new design of confirmation

9669659

Fix underline in confirm link

c2b95a7

Make building of confirmation url more beautiful

34a0474

Minor improvements

bfd03ff

- Add mjml_build script to .PHONY and frontend-deps targets - Make gdg_logo take absolute url - Make build_mjml.sh use `echo -n` instead of `printf` - Remove build html file from diff

Minor improvements

375c4e9

- Fix usage of .PHONY target - Fix `echo -n` issue

cleac force-pushed the master branch from 97148ef to 375c4e9 Compare May 29, 2017 05:32

Alexander Nesterenko and others added 2 commits May 29, 2017 08:45

Fix spaces issues in script running

c7fc292

- Make errors be thrown to stderr

Merge branch 'master' into master

cd6dfe2

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[#37] Update design of confirmation letter #117

[#37] Update design of confirmation letter #117

[#37] Update design of confirmation letter #117

Are you sure you want to change the base?

[#37] Update design of confirmation letter #117

Conversation

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment