Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feature(website): campaign page #725

Merged
merged 6 commits into from
Mar 10, 2024
Merged

feature(website): campaign page #725

merged 6 commits into from
Mar 10, 2024

Conversation

andrashee
Copy link
Contributor

Towards a campaign page. Still needs fine-tuning, but the payment association already works.

No contributions yet:
image

Some contributions:
image

No goals:
image

Last day to contribute:
image

1 day to contribute
image

Todos

  • styling
  • wording
  • one-time contribution: pre-selected amounts
  • monthly payments?

Copy link

vercel bot commented Jan 28, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
public ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 10, 2024 2:57pm

Copy link

github-actions bot commented Jan 28, 2024

PR Preview Action v1.4.7
🚀 Deployed preview to https://socialincome-san.github.io/public/pr-preview/pr-725/
on branch gh-pages at 2024-03-10 14:43 UTC

Copy link

github-actions bot commented Jan 28, 2024

Visit the preview URL for this PR (updated for commit 81a9486):

https://si-admin-staging--pr725-ahee-camaign-page-b2z49cjj.web.app

(expires Sun, 17 Mar 2024 14:57:00 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: b7b0969384059dce6ea8fad1ee1d1737e54e6676

@ssandino
Copy link
Member

Fabulous! Happy to help with text and styling inputs!

Copy link
Contributor

@mkue mkue left a comment

Choose a reason for hiding this comment

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

Very cool stuff @andrashee!

If we want to have this look nicer, I think we should first draw some mockups. For the moment, it looks good to me🙌🏼

</div>
{daysLeft >= 0 && (
<Fragment>
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: I always use <></>. But it's exactly the same...

</BaseContainer>
);
}

const daysUntilTs = (ts: Date): number => {
Copy link
Contributor

Choose a reason for hiding this comment

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

I would move this to /shared/src/utils/date.ts. That's where other date utility functions are located.

@ssandino
Copy link
Member

ssandino commented Feb 1, 2024

Very cool stuff @andrashee!

If we want to have this look nicer, I think we should first draw some mockups. For the moment, it looks good to me🙌🏼

Agree! Very cool. @mkue @andrashee I made some mockups in our Figma file (see page Campaign Page wip). For the moment just ideas about how to place things and what to include (and only for the case, where person chooses a goal) – so it's more starter to discuss, feel free to comment here or directly in the Figma file if easier.

I took two elements from the newly proposed design: background color and FAQ. (new designs from Milk you also find in the Figma)

Here my favourite so far:
Generic Campaign Page

@ssandino
Copy link
Member

ssandino commented Feb 6, 2024

@mkue @andrashee I asked @johanneswicki for input on the mockup. He has come up with another version ("ich habe hier versucht, das ganze etwas grosszügiger zu machen und damit mehr die künftige landing page zu matchen")

If we want to have this look nicer, I think we should first draw some mockups.

Any inputs for a next version?

Screenshot 2024-02-06 at 17 19 54

@andrashee
Copy link
Contributor Author

Thanks a lot @ssandino, the mock-up look great! If you have time tomorrow, we could discuss some details. Would be free in the afternoon.

@andrashee andrashee merged commit 02a771f into main Mar 10, 2024
22 checks passed
@andrashee andrashee deleted the ahee/camaign-page branch March 10, 2024 15:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

3 participants