Waive w22 fees #395

Merged
a258wang merged 3 commits from waive-w22-fees into main 1 year ago
r389li commented 1 year ago
Owner
There is no content yet.
r389li added the
Content
Priority - High
labels 1 year ago
a258wang was assigned by r389li 1 year ago
r389li added 1 commit 1 year ago
be967e96a6 Waive w22 fees
r389li added 1 commit 1 year ago
69c34c78a5 Move paypal button down
r389li requested review from a258wang 1 year ago
a258wang reviewed 1 year ago
You will need to pay the membership fee of $2 through PayPal.
A one-term payment via PayPal comes out to $2.37 (due to PayPal fees).
<s>You will need to pay the membership fee of $2 through PayPal.
A one-term payment via PayPal comes out to $2.37 (due to PayPal fees).</s>
Owner

Could we please change the <s>...</s> to ~~...~~ for the strikethrough instead? The <s> tags are messing up the text margins (https://csclub.uwaterloo.ca/~a3thakra/csc/waive-w22-fees/get-involved/) but changing it to ~~ seems to fix it.

Could we please change the `<s>...</s>` to `~~...~~` for the strikethrough instead? The `<s>` tags are messing up the text margins (https://csclub.uwaterloo.ca/~a3thakra/csc/waive-w22-fees/get-involved/) but changing it to `~~` seems to fix it.
Poster
Owner

Sure, just realized it's markdown, the file uses so much HTML lol

Sure, just realized it's markdown, the file uses so much HTML lol
r389li marked this conversation as resolved
a258wang reviewed 1 year ago
#### Membership Renewal
<b>Membership renewals for the Winter 2022 term are free.</b>
Owner

Similar to above, could we please use **...** for bolding instead of <b>...</b>?

Similar to above, could we please use `**...**` for bolding instead of `<b>...</b>`?
r389li marked this conversation as resolved
a258wang requested changes 1 year ago
a258wang left a comment
Owner

Just a couple of nitpicks, once those are changed we'll be good to go! Thanks 🙂

Just a couple of nitpicks, once those are changed we'll be good to go! Thanks 🙂
r389li added 1 commit 1 year ago
67bf2b620b Use markdown instead of HTML syntax
r389li requested review from a258wang 1 year ago
a258wang approved these changes 1 year ago
a258wang left a comment
Owner

Thank you!

Thank you!
a258wang merged commit b9cd0f6b66 into main 1 year ago
a258wang deleted branch waive-w22-fees 1 year ago
a258wang referenced this issue from a commit 1 year ago

Reviewers

a258wang approved these changes 1 year ago
continuous-integration/drone/push Build is passing
The pull request has been merged as b9cd0f6b66.
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: www/www-new#395
Loading…
There is no content yet.