Skip to content

Conversation

MestiriBaha
Copy link
Contributor

@MestiriBaha MestiriBaha commented May 20, 2024

This PR Closes Issue

closes #74

Description

I have made the necessary adjustments to correct the GraphQL query that retrieves author data from Hashnode's API. The changes are as follows:
- In graphql.operations.ts: Added username and id to the queries that contain author.
- In posts.ts : Added username and included id and username properties in the Author interface.
- In post-details: Changed post.author.name to post.author.username.

What type of PR is this? (check all applicable)

  • 🅰️ Feature
  • 🐛 Bug Fix
  • 📝 Documentation Update
  • 🎨 Style
  • 🧑‍💻 Code Refactor
  • 🔥 Performance Improvements
  • ✅ Test
  • 🤖 Build
  • 🔁 CI
  • 📦 Chore (Release)
  • ⏩ Revert

Mobile & Desktop Screenshots/Recordings

[Attach screenshots or recordings if applicable]

Steps to QA

Added to documentation?

  • 📜 README.md
  • 🙅 no documentation needed

[Optional] Post-deployment tasks

[Specify any post-deployment tasks that need to be performed]

[Optional] What gif best describes this PR or how it makes you feel?

[Embed gif or describe the feeling in plain text]

@MestiriBaha
Copy link
Contributor Author

@cleancodecraft , Hello ! what do you think about the commit & pull request titles , should I adjust something ? thank you !

@esteecodes esteecodes changed the title fix(postDetails,GraphQl-operations) : Inconsistency in GraphQL Query for Author Data fix(post-details) : Inconsistency in GraphQL Query for Author Data May 22, 2024
@esteecodes esteecodes added 🐛 bug Something isn't working GraphGL labels May 22, 2024
Copy link
Member

@esteecodes esteecodes left a comment

Choose a reason for hiding this comment

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

@MestiriBaha everything looks great
thank you for your contribution 🎉

@esteecodes esteecodes merged commit 48b714d into HashnodeWithAngular:develop May 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working GraphGL
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug: Inconsistency in GraphQL Query for Author Data
2 participants