Skip to content

Conversation

alexhking
Copy link
Contributor

Adding aside to new issue modal with Discord link, messaging.

Copy link

vercel bot commented Sep 9, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
replicache-docs Ready Ready Preview Comment Sep 9, 2025 8:04pm
zbugs Ready Ready Preview Comment Sep 9, 2025 8:04pm

Copy link

github-actions bot commented Sep 9, 2025

🐰 Bencher Report

Branchalex/new-issue-discord-message
TestbedLinux
Click to view all benchmark results
BenchmarkFile SizeBenchmark Result
kilobytes (KB)
(Result Δ%)
Upper Boundary
kilobytes (KB)
(Limit %)
zero-package.tgz📈 view plot
🚷 view threshold
1,310.94 KB
(0.00%)Baseline: 1,310.94 KB
1,337.16 KB
(98.04%)
zero.js📈 view plot
🚷 view threshold
209.78 KB
(0.00%)Baseline: 209.78 KB
213.97 KB
(98.04%)
zero.js.br📈 view plot
🚷 view threshold
58.91 KB
(0.00%)Baseline: 58.91 KB
60.09 KB
(98.04%)
🐰 View full continuous benchmarking report in Bencher

Copy link

github-actions bot commented Sep 9, 2025

🐰 Bencher Report

Branchalex/new-issue-discord-message
Testbedself-hosted
Click to view all benchmark results
BenchmarkThroughputBenchmark Result
operations / second (ops/s) x 1e3
(Result Δ%)
Lower Boundary
operations / second (ops/s) x 1e3
(Limit %)
src/client/custom.bench.ts > big schema📈 view plot
🚷 view threshold
841.28 ops/s x 1e3
(+11.63%)Baseline: 753.62 ops/s x 1e3
518.69 ops/s x 1e3
(61.66%)
src/client/zero.bench.ts > basics > All 1000 rows x 10 columns (numbers)📈 view plot
🚷 view threshold
2.51 ops/s x 1e3
(-5.57%)Baseline: 2.66 ops/s x 1e3
2.50 ops/s x 1e3
(99.88%)
src/client/zero.bench.ts > pk compare > pk = N📈 view plot
🚷 view threshold
42.38 ops/s x 1e3
(+0.53%)Baseline: 42.16 ops/s x 1e3
40.43 ops/s x 1e3
(95.39%)
src/client/zero.bench.ts > with filter > Lower rows 500 x 10 columns (numbers)📈 view plot
🚷 view threshold
4.13 ops/s x 1e3
(+2.26%)Baseline: 4.04 ops/s x 1e3
3.83 ops/s x 1e3
(92.67%)
🐰 View full continuous benchmarking report in Bencher

@aboodman
Copy link
Contributor

CleanShot 2025-09-09 at 19 49 59@2x

Should we also have something like this in the comment box?

@alexhking
Copy link
Contributor Author

CleanShot 2025-09-09 at 19 49 59@2x
  1. The capitalization makes sense. Sometimes we style zero-related things as lowercase so I had it this way, but I think it makes more sense to be capitalized.
  2. I think "issue" is the right word to use.
  3. I can update and show you what it looks like for CTA to be a link with no button

Should we also have something like this in the comment box?

Yeah, we can add this. I just don't want things to be too noisy. In the comment area I think we should just be showing the second sentence: "Want a faster response? Join us on Discord."

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