devrsi0n commented 1 year ago

Description

Fixes a dozen of issues:

Can't post a comment in comment widget preview on home page
Shouldn't query notification messages in comment widget preview

Added a new feature:

show popover if user can't create a project, e.g. system maintenance mode, anonymous user or reach maximum project number

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Checklist

[ ] Unit tests and integration tests passed
[ ] No reduction in test coverage
[ ] Documentation is up to date (if appropriate)
[ ] Related issues linked using fixes #number

changeset-bot[bot] commented 1 year ago

⚠️ No Changeset found

Latest commit: 7426e220c2efd1f0e3dde65143d13d1a5739bb39

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

vercel[bot] commented 1 year ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated
chirpy	✅ Ready (Inspect)	Visit Preview	Dec 9, 2022 at 2:26PM (UTC)

github-actions[bot] commented 1 year ago

Coverage report for `./apps/main`

St.^{:grey_question:}	Category	Percentage	Covered / Total
🔴	Statements	8.73%	40/458
🔴	Branches	10.23%	9/88
🔴	Functions	4.1%	5/122
🔴	Lines	9.25%	38/411

Test suite run success

2 tests passing in 2 suites.

Report generated by 🧪jest coverage report action from 7426e220c2efd1f0e3dde65143d13d1a5739bb39

github-actions[bot] commented 1 year ago

Coverage report for `./packages/ui`

St.^{:grey_question:}	Category	Percentage	Covered / Total
🔴	Statements	52.85% (+0.13% 🔼)	2373/4490
🔴	Branches	26.89% (-0.1% 🔻)	441/1640
🔴	Functions	36.41% (+0.21% 🔼)	461/1266
🔴	Lines	52.95% (+0.14% 🔼)	2256/4261

Show new covered files 🐣

|

St.^{:grey_question:}

| File | Statements | Branches | Functions | Lines | | :----------------------------------------------------------------------------------------------------------------------------------------------------: | :--------------------------------------------------------------------------------------------- | :--------- | :------- | :-------- | :----- | | 🟡 |

`...` / create-project-button.tsx

| 78.57% | 36.36% | 100% | 78.57% |

Show files with reduced coverage 🔻

|

St.^{:grey_question:}

| File | Statements | Branches | Functions | Lines | | :----------------------------------------------------------------------------------------------------------------------------------------------------: | :------------------------------------------------------------------------------------------------------------- | :------------------------------------------------------------- | :------------------------------------------------------------- | :------------------------------------------------------------ | :------------------------------------------------------------- | | 🟢 |

`...` / user.ts

|

85.71% (-14.29% 🔻)

| 100% |

66.67% (-33.33% 🔻)

| 100% | | 🟡 |

`...` / index.tsx

|

64.29% (-1.23% 🔻)

|

57.14% (+3.3% 🔼)

| 40% |

64.29% (-1.23% 🔻)

| | 🟢 |

`...` / notification-hub.tsx

|

92.31% (+3.85% 🔼)

|

84.62% (-6.29% 🔻)

| 100% |

92% (+4% 🔼)

| | 🔴 |

`...` / comment-widget-preview.tsx

| 34.55% |

10% (-8.18% 🔻)

| 25% | 35.85% |

Test suite run success

50 tests passing in 32 suites.

Report generated by 🧪jest coverage report action from 7426e220c2efd1f0e3dde65143d13d1a5739bb39

relativeci[bot] commented 1 year ago

Job #1209: Bundle Size — 2.02MiB (0%).

1510937(current) vs 1510937 main#1208(baseline)

Metrics (1 change)

	Current Job #1209	Baseline Job #1208
Initial JS	`1.39MiB`	`1.39MiB`
Initial CSS	`90.41KiB`	`90.41KiB`
Cache Invalidation	`0%`	`24.42%`
Chunks	`50`	`50`
Assets	`57`	`57`
Modules	`1045`	`1045`
Duplicate Modules	`182`	`182`
Duplicate Code	`4.83%`	`4.83%`
Packages	`88`	`88`
Duplicate Packages	`0`	`0`

Total size by type (no changes)

| | Current
[Job #1209](https://app.relative-ci.com/projects/rYoEVFddayylGRgFiBEd/jobs/1209-kH7bmDCFFVXyKWhh47GO?utm_source=github&utm_medium=pr-report "View job report") | Baseline
[Job #1208](https://app.relative-ci.com/projects/rYoEVFddayylGRgFiBEd/jobs/1208-uqi0yPlhOgOmojnlGklF?utm_source=github&utm_medium=pr-report "View baseline job report") | |:--|--:|--:| | [CSS](https://app.relative-ci.com/projects/rYoEVFddayylGRgFiBEd/jobs/1209-kH7bmDCFFVXyKWhh47GO/assets?ba=%7B%22filters%22%3A%7B%22ft.CSS%22%3Atrue%2C%22ft.JS%22%3Afalse%2C%22ft.IMG%22%3Afalse%2C%22ft.MEDIA%22%3Afalse%2C%22ft.FONT%22%3Afalse%2C%22ft.HTML%22%3Afalse%2C%22ft.OTHER%22%3Afalse%7D%7D "View all CSS assets") | `90.41KiB` | `90.41KiB` | | [Fonts](https://app.relative-ci.com/projects/rYoEVFddayylGRgFiBEd/jobs/1209-kH7bmDCFFVXyKWhh47GO/assets?ba=%7B%22filters%22%3A%7B%22ft.CSS%22%3Afalse%2C%22ft.JS%22%3Afalse%2C%22ft.IMG%22%3Afalse%2C%22ft.MEDIA%22%3Afalse%2C%22ft.FONT%22%3Atrue%2C%22ft.HTML%22%3Afalse%2C%22ft.OTHER%22%3Afalse%7D%7D "View all Fonts assets") | `0B` | `0B` | | [HTML](https://app.relative-ci.com/projects/rYoEVFddayylGRgFiBEd/jobs/1209-kH7bmDCFFVXyKWhh47GO/assets?ba=%7B%22filters%22%3A%7B%22ft.CSS%22%3Afalse%2C%22ft.JS%22%3Afalse%2C%22ft.IMG%22%3Afalse%2C%22ft.MEDIA%22%3Afalse%2C%22ft.FONT%22%3Afalse%2C%22ft.HTML%22%3Atrue%2C%22ft.OTHER%22%3Afalse%7D%7D "View all HTML assets") | `0B` | `0B` | | [IMG](https://app.relative-ci.com/projects/rYoEVFddayylGRgFiBEd/jobs/1209-kH7bmDCFFVXyKWhh47GO/assets?ba=%7B%22filters%22%3A%7B%22ft.CSS%22%3Afalse%2C%22ft.JS%22%3Afalse%2C%22ft.IMG%22%3Atrue%2C%22ft.MEDIA%22%3Afalse%2C%22ft.FONT%22%3Afalse%2C%22ft.HTML%22%3Afalse%2C%22ft.OTHER%22%3Afalse%7D%7D "View all IMG assets") | `0B` | `0B` | | [JS](https://app.relative-ci.com/projects/rYoEVFddayylGRgFiBEd/jobs/1209-kH7bmDCFFVXyKWhh47GO/assets?ba=%7B%22filters%22%3A%7B%22ft.CSS%22%3Afalse%2C%22ft.JS%22%3Atrue%2C%22ft.IMG%22%3Afalse%2C%22ft.MEDIA%22%3Afalse%2C%22ft.FONT%22%3Afalse%2C%22ft.HTML%22%3Afalse%2C%22ft.OTHER%22%3Afalse%7D%7D "View all JS assets") | `1.92MiB` | `1.92MiB` | | [Media](https://app.relative-ci.com/projects/rYoEVFddayylGRgFiBEd/jobs/1209-kH7bmDCFFVXyKWhh47GO/assets?ba=%7B%22filters%22%3A%7B%22ft.CSS%22%3Afalse%2C%22ft.JS%22%3Afalse%2C%22ft.IMG%22%3Afalse%2C%22ft.MEDIA%22%3Atrue%2C%22ft.FONT%22%3Afalse%2C%22ft.HTML%22%3Afalse%2C%22ft.OTHER%22%3Afalse%7D%7D "View all Media assets") | `0B` | `0B` | | [Other](https://app.relative-ci.com/projects/rYoEVFddayylGRgFiBEd/jobs/1209-kH7bmDCFFVXyKWhh47GO/assets?ba=%7B%22filters%22%3A%7B%22ft.CSS%22%3Afalse%2C%22ft.JS%22%3Afalse%2C%22ft.IMG%22%3Afalse%2C%22ft.MEDIA%22%3Afalse%2C%22ft.FONT%22%3Afalse%2C%22ft.HTML%22%3Afalse%2C%22ft.OTHER%22%3Atrue%7D%7D "View all Other assets") | `11.53KiB` | `11.53KiB` |

View job #1209 report View prod branch activity

cypress[bot] commented 1 year ago

Test summary

4 • 2 • 0 • 1 • 0

Run details


Project	chirpy
Status	Failed
Commit	7426e220c2
Started	Dec 9, 2022 2:27 PM
Ended	Dec 9, 2022 2:30 PM
Duration	02:20 💡
OS	Linux Ubuntu - 22.04
Browser	Chrome 108

View run in Cypress Dashboard ➡️

Failures

	home/header.spec.ts	1
1	Header > should show user menu
	dashboard/project.spec.ts	1
1	Project > should show integration doc

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

devrsi0n / chirpy

fix(comment-preview): can't post a comment and don't query notification messages if user doesn't signin #553

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Checklist

⚠️ No Changeset found

Coverage report for `./apps/main`

Test suite run success

Coverage report for `./packages/ui`

Test suite run success

Job #1209: Bundle Size — 2.02MiB (0%).

Test summary

Run details

Failures

devrsi0n / chirpy

fix(comment-preview): can't post a comment and don't query notification messages if user doesn't signin #553

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Checklist

⚠️ No Changeset found

Coverage report for ./apps/main

Test suite run success

Coverage report for ./packages/ui

Test suite run success

Job #1209: Bundle Size — 2.02MiB (0%).

Test summary

Run details

Failures

Coverage report for `./apps/main`

Coverage report for `./packages/ui`