website icon indicating copy to clipboard operation
website copied to clipboard

[BUG] Newsletter page does not have correct button labels

Open JeelRajodiya opened this issue 1 year ago • 10 comments

Describe the bug.

The labels for the buttons and input fields simply have the i18n placeholders instead of the actual labels

image image

Expected behavior

These are the expected labels

image

How to Reproduce

  1. Go to https://www.asyncapi.com/newsletter or https://www.asyncapi.com/community/tsc
  2. You will notice the improper labels

🥦 Browser

Google Chrome

👀 Have you checked for similar open issues?

  • [X] I checked and didn't find similar issue

🏢 Have you read the Contributing Guidelines?

Are you willing to work on this issue ?

No, someone else can work on it

JeelRajodiya avatar Sep 15 '24 15:09 JeelRajodiya

I can work on it!

jayp5545 avatar Sep 15 '24 17:09 jayp5545

How do we access newsletter page?

sambhavgupta0705 avatar Sep 17 '24 08:09 sambhavgupta0705

How do we access newsletter page?

I am not sure either, I was just going through the codebase and found that we have pages/newsletter.tsx file. Which means the page exists at https://www.asyncapi.com/newsletter

image

JeelRajodiya avatar Sep 17 '24 08:09 JeelRajodiya

@akshatnema do we need this url as we only have newsletter at the home page which is working fine

sambhavgupta0705 avatar Sep 17 '24 08:09 sambhavgupta0705

@sambhavgupta0705 look at 'components/NewsletterSubscribe.tsx'

hkv24 avatar Sep 17 '24 16:09 hkv24

Hey,@JeelRajodiya can you assign this to me.

prashantrai-30 avatar Sep 18 '24 05:09 prashantrai-30

Found it. Here is how you can access the /newsletter page

1. Go to the footer of the website.

2. Click on About

image

3. Now on the about page. you will find a link to the newsletter

image

That's not it. The newsletter page is also linked from multiple blogs

JeelRajodiya avatar Sep 18 '24 08:09 JeelRajodiya

Hey,@JeelRajodiya can you assign this to me.

Issues are not assigned on the website repo (AFAIK). If the issue is valid, you can start working on it and a create PR.

JeelRajodiya avatar Sep 18 '24 08:09 JeelRajodiya

@JeelRajodiya I can still see this issue present on the website.Can i work on this ?

Aryan4884 avatar Oct 23 '24 09:10 Aryan4884

@JeelRajodiya I can still see this issue present on the website.Can i work on this ?

I think PR https://github.com/asyncapi/website/pull/3308 is relevant to this issue and is still under review. You should avoid working on the issues which already have a PR open.

JeelRajodiya avatar Oct 24 '24 10:10 JeelRajodiya

Reopening this one

sambhavgupta0705 avatar Dec 16 '24 15:12 sambhavgupta0705

@sambhavgupta0705 I think the issue was resolved in #3308. We can close this issue.

JeelRajodiya avatar Dec 19 '24 10:12 JeelRajodiya