Subscribe button on homepage

[sohaibaumer]

Fixes #598

• Renamed the branch/commit as per requirement.

• Screenshot is also attached:
  

[MaxGhenis]

Thanks - looks nice. Could you please:

• Zoom out for the screenshot so we can see where the content below is
• Add a mobile screenshot
• Remove Fixes #598 from the PR title and move it into the PR description (the first comment, which you can edit). You'll know you've correctly linked it when it shows up in the Development section on the right sidebar

[sohaibaumer]

Fixes #598

Desktop Screenshot:

Mobile Screenshot:

[nikhilwoodruff]

Thanks @sohaibaumer - looks great. I made a few style changes (fixing centering and removing the top white gap). I couldn't immediately figure out the test issue though, do you have any ideas what changed?

[sohaibaumer]

@nikhilwoodruff
not sure, I re-ran the test and they pass.

PS, I didn't change much in code anyways, re-used one of your existing functions.

[MaxGhenis]

Any update here? Could you try pulling in the latest changes to see if that affects CI?

[MaxGhenis]

What's the status on this?

[anth-volk]

I wrote a PR last week that creates this functionality, but it could probably be made more robust with front-end authentication practices in the future. I believe Nikhil has pushed the code into prod.

…

On Dec 10, 2023, at 11:18, Max Ghenis ***@***.***> wrote:  What's the status on this? — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you are subscribed to this thread.

[anth-volk]

Closing, as this functionality was added with #924.