domain linking form #5

Merged
muppeth merged 6 commits from domain-linking-form into master 2019-10-16 19:15:00 +02:00
Owner
No description provided.
antilopa was assigned by fede 2019-10-12 00:19:59 +02:00
muppeth was assigned by fede 2019-10-12 00:19:59 +02:00
meaz was assigned by fede 2019-10-12 00:19:59 +02:00
fede self-assigned this 2019-10-12 00:19:59 +02:00
maryjane was assigned by fede 2019-10-12 00:19:59 +02:00
fede added the
REVISION
FORM
labels 2019-10-12 00:19:59 +02:00
Owner

For "amount" we should perhaps add a list of currency that users could choose from. I remember in tickets that sometimes we know that a user gave 5, but we don't know if it's in $, € or whatever currency they use.

For "amount" we should perhaps add a list of currency that users could choose from. I remember in tickets that sometimes we know that a user gave 5, but we don't know if it's in $, € or whatever currency they use.
Owner

Also, for placeholder: 'Name of payment holder or other transfer reference', the text is a bit long, so we don't see it all.

Also, for placeholder: 'Name of payment holder or other transfer reference', the text is a bit long, so we don't see it all.
Author
Owner

Yes, it is indeed a bit long, though I'm not sure about how to make it short and descriptive enough.

Yes, it is indeed a bit long, though I'm not sure about how to make it short and descriptive enough.
Owner

perhaps it ould be set under the title of the box, not in the box. ANd in the ox, we just write "Reference of payment"

perhaps it ould be set under the title of the box, not in the box. ANd in the ox, we just write "Reference of payment"
Owner

I just pushed some changes to the form text. Please review.
(screenshot attached)

I just pushed some changes to the form text. Please review. (screenshot attached)
Owner

seems fine to me

seems fine to me
meaz approved these changes 2019-10-14 01:10:25 +02:00
meaz left a comment
Owner

perhaps there are a bit too many

perhaps there are a bit too many <br>
Author
Owner

I think we can merge it.

I think we can merge it.
Owner

I like what you did with the instruction steps muppeth. Let's add on the '2. start donating' the sentence - to see how to donate go to https:disroot.org/donate

I like what you did with the instruction steps muppeth. Let's add on the '2. start donating' the sentence - to see how to donate go to https:disroot.org/donate
Owner

like this? (check latest commit or attachement)

like this? (check latest commit or attachement)
Owner

Waiting for one more approval before merging to master.

Waiting for one more approval before merging to master.
antilopa approved these changes 2019-10-16 16:31:53 +02:00
muppeth referenced this issue from a commit 2019-10-16 19:15:00 +02:00
muppeth closed this pull request 2019-10-16 19:15:00 +02:00
fede referenced this issue from a commit 2022-06-16 06:08:21 +02:00
fede referenced this issue from a commit 2022-06-17 12:02:17 +02:00
Sign in to join this conversation.
No description provided.