Skip to content

feat(comment): hide most of info in archived/banned comments#3757

Open
gary02 wants to merge 1 commit into
developfrom
feat/hide-archived-comment-info
Open

feat(comment): hide most of info in archived/banned comments#3757
gary02 wants to merge 1 commit into
developfrom
feat/hide-archived-comment-info

Conversation

@gary02

@gary02 gary02 commented Jan 12, 2024

Copy link
Copy Markdown
Contributor

No description provided.

@gary02 gary02 requested a review from a team as a code owner January 12, 2024 14:05
@codecov

codecov Bot commented Jan 12, 2024

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 82.35294% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 58.30%. Comparing base (9ddd7bd) to head (aee1516).
⚠️ Report is 2445 commits behind head on develop.

Files with missing lines Patch % Lines
src/queries/comment/fromDonator.ts 0.00% 9 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #3757      +/-   ##
===========================================
+ Coverage    58.21%   58.30%   +0.08%     
===========================================
  Files          866      867       +1     
  Lines        17175    17216      +41     
  Branches      3163     3172       +9     
===========================================
+ Hits          9999    10038      +39     
- Misses        6495     6498       +3     
+ Partials       681      680       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@gary02 gary02 force-pushed the feat/hide-archived-comment-info branch from e6786cf to aee1516 Compare January 12, 2024 14:45
return createdAt
} else {
// invalid date
return new Date(0)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

should return null instead?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

createdAt: DateTime!
can not return null here without update schema

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