Skip to content

Refactor: Add SSR and API error handling; UI/data fixes; API docs#53

Closed
SaptarshiMondal123 wants to merge 9 commits intoAvishkarPatil:mainfrom
SaptarshiMondal123:main
Closed

Refactor: Add SSR and API error handling; UI/data fixes; API docs#53
SaptarshiMondal123 wants to merge 9 commits intoAvishkarPatil:mainfrom
SaptarshiMondal123:main

Conversation

@SaptarshiMondal123
Copy link

@SaptarshiMondal123 SaptarshiMondal123 commented Aug 29, 2025

📝 Description

This pull request includes several key updates, including new features, bug fixes, and documentation improvements. My goal is to enhance functionality and maintainability while addressing existing issues.

What's Changed:

  • New Features:
    • Added SSR (Server-Side Rendering) compatibility to the ArticleActions component.
  • Refactoring:
    • Standardized the API error-handling logic for a more consistent and robust approach.
  • Bug Fixes:
    • Fixed an error in data.ts to ensure data is displayed correctly.
    • Enabled middleware authentication protection to secure certain routes.
  • Documentation:
    • Added a new section to the API documentation to clarify the difference between using id and slug in API endpoints.

🔗 Related Issue

N/A (or link to a specific issue if you were solving one)

🔄 Type of Change

  • ✨ New feature (non-breaking change which adds functionality)
  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • 📚 Documentation update

✅ Checklist

  • My code follows the project's coding standards.
  • I have updated documentation where necessary.
  • My changes generate no new warnings.

@github-actions
Copy link

Thanks for creating a PR for your Issue! ☺️

We'll review it as soon as possible.
In the meantime, please double-check the file changes and ensure that all commits are accurate.

If there are any unresolved review comments, feel free to resolve them. 🙌🏼

@SaptarshiMondal123 SaptarshiMondal123 changed the title Fix: UI and data errors, and update API docs Refactor: Add SSR and API error handling; UI/data fixes; API docs Aug 30, 2025
@AvishkarPatil
Copy link
Owner

Thanks @SaptarshiMondal123, but this PR appears to be created for contribution stats rather than solving real issues.

Problems:

  • Just uncommented existing code
  • Multiple commits for same changes
  • Debug logs left in production code
  • No clear problem being solved

Please focus on quality contributions that address actual issues rather than quantity.

❌ Closing this PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants