Skip to content

Feat/blog from hashnode#63

Merged
priyankarpal merged 20 commits into
reactplay:mainfrom
Ravijhabit:feat/blog-from-hashnode
Oct 11, 2025
Merged

Feat/blog from hashnode#63
priyankarpal merged 20 commits into
reactplay:mainfrom
Ravijhabit:feat/blog-from-hashnode

Conversation

@Ravijhabit
Copy link
Copy Markdown
Contributor

Description: Integrate Hashnode and fetch articles from hashnode.

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

  • Created a blog component (Because it would be used at /blog path and blog section in homepage)
  • Removed blog component from /module/(section)
  • Declared function to fetch blogs from hashnode: initialBlogFunction and server-action function (for additional blogs in /blog page)
  • Mapped the response with prop to have ui consistency with current design.
  • Declared type for Hashnode response and blog type

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • [✅ ] New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

Manually tested
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce.

Checklist:

  • [ ✅ ] I have performed a self-review of my own code
  • [ ✅] I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • [ ✅] My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

Screenshots or example output

Homepage:
image

  • Filters are working
  • View all posts leads to /blog page

Blog:
image

  • Filters are working here as well
  • View all posts CTA is not available
  • Load more button is available(fetches more blogs from hashnode)

@netlify
Copy link
Copy Markdown

netlify Bot commented Oct 7, 2025

Deploy Preview for reactkolkata ready!

Name Link
🔨 Latest commit 7abd3e0
🔍 Latest deploy log https://app.netlify.com/projects/reactkolkata/deploys/68e9ed6987c63c0008c8401c
😎 Deploy Preview https://deploy-preview-63--reactkolkata.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Copy Markdown
Member

@priyankarpal priyankarpal left a comment

Choose a reason for hiding this comment

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

can you check this?

Image

also there are some ui issues.

Image

just change the width for desktop/laptop also remove the scroll bar
Image

Copy link
Copy Markdown
Member

@priyankarpal priyankarpal left a comment

Choose a reason for hiding this comment

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

also can we add error message display ui? @Ravijhabit

Comment thread src/components/common/blog/BlogList.tsx Outdated
Comment thread src/components/common/blog/BlogList.tsx Outdated
@priyankarpal
Copy link
Copy Markdown
Member

removed resources, get in touch from the website you can see also can you make the blog card clickable?

@Ravijhabit
Copy link
Copy Markdown
Contributor Author

@priyankarpal, I am working on the comment but you are adding new requirements like

  1. Removed resources: Why? (What are we trying to achieve here with this change)
  2. get in touch from the website: Why? (Please share the intent for this change)
  3. make the blog card clickable: Why? (We already have Read More button which opens the blog in modal format)
    Please share the intent along with the changes.

@Ravijhabit
Copy link
Copy Markdown
Contributor Author

Ravijhabit commented Oct 8, 2025

@priyankarpal I had a question on blogModal. What are we trying to achieve here?
Should we show the entire blog in blogModal itself?

Except blog modal I have resolve all the other issues. Would be working on blogModal once I have clarity on blogModal.

Right now, blogModal shows contain that is already present in the blogCard (No extra content)
If we add entire blog detail in the modal, then do we require the redirection_url to react-dev.hashnode page?

@priyankarpal
Copy link
Copy Markdown
Member

@priyankarpal, I am working on the comment but you are adding new requirements like

  1. Removed resources: Why? (What are we trying to achieve here with this change)
  2. get in touch from the website: Why? (Please share the intent for this change)
  3. make the blog card clickable: Why? (We already have Read More button which opens the blog in modal format)
    Please share the intent along with the changes.

ok so i removed these, we had a discussion with core team and decided to remove few things(not required right now) from the website & add new sections(already created few issues)

  1. Removed resources: Why? (What are we trying to achieve here with this change)
  2. get in touch from the website: Why? (Please share the intent for this change)

Yeah, there’s a ‘Read More’ button, but most users including myself don’t click it they usually click on the blog title instead. :)

  1. make the blog card clickable: Why? (We already have Read More button which opens the blog in modal format)

@priyankarpal
Copy link
Copy Markdown
Member

@Ravijhabit No need to add anything to the blog modal, just a small UI fix is needed. When you click the ‘Read More’ button, the popup appears a bit narrow it would look better if it’s little wider

@priyankarpal
Copy link
Copy Markdown
Member

@Ravijhabit just a small fix here (1920x1080)

image

@priyankarpal
Copy link
Copy Markdown
Member

@Ravijhabit any update?

@Ravijhabit
Copy link
Copy Markdown
Contributor Author

@Ravijhabit any update?

Working on the fix. Modal might seems okay right now but it contains hard-coded content for Key Highlights and What you'll learn
image

What should we do for those?
Sorry for the delay.

@priyankarpal
Copy link
Copy Markdown
Member

@Ravijhabit any update?

Working on the fix. Modal might seems okay right now but it contains hard-coded content for Key Highlights and What you'll learn image

What should we do for those? Sorry for the delay.

remove hard coded parts

@Ravijhabit
Copy link
Copy Markdown
Contributor Author

@priyankarpal Please review

@priyankarpal priyankarpal merged commit 22c4f3d into reactplay:main Oct 11, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants