Skip to content

Update Readme.md#2408

Open
wanda-carlson wants to merge 1 commit intomainfrom
wanda-carlson-patch-460282
Open

Update Readme.md#2408
wanda-carlson wants to merge 1 commit intomainfrom
wanda-carlson-patch-460282

Conversation

@wanda-carlson
Copy link
Contributor

@wanda-carlson wanda-carlson commented Jan 13, 2025

@wanda-carlson
Copy link
Contributor Author

wanda-carlson commented Jan 21, 2025

  1. Run the script: CLICKHOUSE_URL=http://default:${password}@localhost:40544 deno run --allow-env --allow-net --allow-read

@wanda-carlson
Copy link
Contributor Author

wanda-carlson commented Jan 22, 2025

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

Name Status Preview Comments Updated (UTC)
pullflow-website ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 20, 2025 5:26am

1 similar comment
@wanda-carlson
Copy link
Contributor Author

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

Name Status Preview Comments Updated (UTC)
pullflow-website ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 20, 2025 5:26am

@wanda-carlson
Copy link
Contributor Author

wanda-carlson commented Jan 22, 2025

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

Name Status Preview Comments Updated (UTC)
pullflow-website ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 20, 2025 5:29am

@wanda-carlson
Copy link
Contributor Author

wanda-carlson commented Jan 22, 2025

1 file(s) reviewed, no comment(s)

Edit PR Review Bot Settings - Greptile

@wanda-carlson
Copy link
Contributor Author

wanda-carlson commented Jan 22, 2025

Summary of changes

This pull request removes !user ID from userStats so users can't get stats for other users through URL manipulation.

Changes to GraphQL schema and service logic:

Updates to components:

Config impact

None

Screenshots

image

(verified on local)

Affected core tests

  • Suite: Smoke Test (web app dashboard)

PR-specific tests

  • PR stats in webapp dashboard should show as normal.

@wanda-carlson
Copy link
Contributor Author

wanda-carlson commented Jan 22, 2025

Summary of changes

This pull request removes !user ID from userStats so users can't get stats for othesr users through URL manipulation.

Changes to GraphQL schema and service logic:

Updates to components:

Config impact

None

Screenshots

image

(verified on local)

Affected core tests

  • Suite: Smoke Test (web app dashboard)

PR-specific tests

  • PR stats in webapp dashboard should show as normal.

@wanda-carlson
Copy link
Contributor Author

wanda-carlson commented Jan 25, 2025

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

Name Status Preview Comments Updated (UTC)
pullflow-website ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 26, 2025 0:42am

Copy link
Contributor Author

Synced with Slack by @pullflow

2 similar comments
Copy link
Contributor Author

Synced with Slack by @pullflow

Copy link
Contributor Author

Synced with Slack by @pullflow

Copy link
Contributor Author

Synced at Pullflow setup

Copy link
Contributor Author

Synced at PullFlow setup

Copy link
Contributor Author

🤖 Automated comment from PullFlow setup.

1 similar comment
Copy link
Contributor Author

🤖 Automated comment from PullFlow setup.

@wanda-carlson
Copy link
Contributor Author

hello

@sonarqubecloud
Copy link

Copy link
Contributor Author

🤖 Automated comment from PullFlow setup.

2 similar comments
Copy link
Contributor Author

🤖 Automated comment from PullFlow setup.

Copy link
Contributor Author

🤖 Automated comment from PullFlow setup.

Copy link
Contributor Author

henry review

Copy link
Contributor Author

bert review

Copy link
Collaborator

yes

Copy link
Contributor Author

@henry-richard?

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