Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: Add Comments tab to User Profile #653

Merged
merged 4 commits into from
Apr 7, 2022

Conversation

LulaV14
Copy link
Member

@LulaV14 LulaV14 commented Apr 4, 2022

image

@LulaV14 LulaV14 requested a review from a team as a code owner April 4, 2022 10:02
const userComments = await Comment.query()
.where('authorId', user.userId)
.innerJoin('insight', 'comment.insightId', 'insight.insightId')
.whereNull('insight.deletedAt')
Copy link
Contributor

Choose a reason for hiding this comment

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

Probably need to add .whereNull('comment.deletedAt') here.

cursor: String!
cursor: String
Copy link
Contributor

Choose a reason for hiding this comment

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

How did this change happen?

Copy link
Member Author

Choose a reason for hiding this comment

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

This was probably done when I was testing, but I'm not sure why it didn't revert back.

@@ -86,7 +86,7 @@ export const InsightList = ({
);
}

if (insightConnection.edges.length === 0) {
if (insightConnection.edges?.length === 0) {
Copy link
Contributor

Choose a reason for hiding this comment

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

edges are not nullable, so not sure why this is needed?

Copy link
Member Author

Choose a reason for hiding this comment

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

you're right, I'm not sure how I suddenly got an error from here, seems like insightConnection shouldn't be undefined either at this point.

But I couldn't replicate it anymore, so maybe I'll just remove it from here and create a ticket if it happens again

@LulaV14 LulaV14 requested a review from baumandm April 4, 2022 20:23
Copy link
Contributor

@baumandm baumandm left a comment

Choose a reason for hiding this comment

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

Looks good but there's still one lint warning in user.service.ts

@LulaV14 LulaV14 changed the title fix: Add Comments tab to User Profile feat: Add Comments tab to User Profile Apr 7, 2022
@LulaV14 LulaV14 merged commit dc3db70 into main Apr 7, 2022
@LulaV14 LulaV14 deleted the feature/add-comments-section-on-user-profile branch April 7, 2022 20:36
eg-oss-ci added a commit that referenced this pull request Apr 7, 2022
## [3.8.0](v3.7.0...v3.8.0) (2022-04-07)

### Features

* Add Comments tab to User Profile ([#653](#653)) ([dc3db70](dc3db70))
@eg-oss-ci
Copy link
Contributor

🎉 This PR is included in version 3.8.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Development

Successfully merging this pull request may close these issues.

3 participants