Skip to content

Conversation

ryanrozich
Copy link
Owner

Summary

Updates the add-pr-to-project.yml workflow to use PROJECT_ACCESS_TOKEN instead of the default GITHUB_TOKEN to fix the "Resource not accessible by integration" error.

Problem

The workflow was failing with this error when trying to add PRs to the GitHub Project:

GraphqlResponseError: Request failed due to following response errors:
 - Resource not accessible by integration

This happens because the default GITHUB_TOKEN doesn't have permissions to access organization-level GitHub Projects (Projects V2).

Solution

Changed the token from:

github-token: ${{ secrets.GITHUB_TOKEN }}

To:

github-token: ${{ secrets.PROJECT_ACCESS_TOKEN }}

Setup Required

Before this will work, you need to:

  1. Create a Personal Access Token:

    • Go to https://github.com/settings/tokens/new
    • Name it something like "AG Grid Project Access"
    • Select these scopes:
      • repo (for repository access)
      • project (for project access)
    • Generate and copy the token
  2. Add it as a Repository Secret:

Testing

After adding the secret:

  1. Re-run the failed workflow on PR Archive failing E2E tests to achieve green CI status #86
  2. It should successfully add the PR to the project
  3. All future PRs will be automatically added to the project

Note

Only this one workflow needs the special token. All other workflows continue to use the default GITHUB_TOKEN as they don't need project access.

🤖 Generated with Claude Code

ryanrozich and others added 3 commits July 9, 2025 21:55
- Replace .ag-root with .ag-root-wrapper in filter-presets tests
- Fix AGGridTestHarness to properly register grid API using onGridReady
- Add /test-demo route for E2E testing
- Update TestDemo tests to use grid API directly instead of UI interactions
- Fix column selectors and date filtering logic
- All 5 TestDemo tests now passing
- All 4 Avatar tests remain passing
- Archived 73 failing E2E tests to tests/e2e/archived/ directory
- Created playwright.config.ts to exclude archived tests from runs
- Kept only 9 passing tests active (TestDemo: 5, avatars: 4)
- Added comprehensive README in archived directory with restoration plan
- Updated E2E test analysis report with gaps and future instructions

This establishes a green CI baseline while preserving valuable test coverage
for future restoration. Each archived test is documented with its failure
reason and fix requirements.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
The default GITHUB_TOKEN doesn't have permissions to access organization-level
GitHub Projects (Projects V2). This change updates the workflow to use a
personal access token with the necessary project permissions.

To fix the "Resource not accessible by integration" error, you need to:
1. Create a PAT with 'repo' and 'project' scopes
2. Add it as PROJECT_ACCESS_TOKEN in repository secrets
3. Re-run the failed workflow

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@github-actions github-actions bot added the status: in-code-review PR ready, awaiting code review label Jul 10, 2025
@ryanrozich ryanrozich merged commit 318057c into release/v0.2.0-rc2 Jul 10, 2025
13 of 14 checks passed
@github-actions github-actions bot added status: merged PR merged to main branch and removed status: in-code-review PR ready, awaiting code review labels Jul 10, 2025
Copy link

🧹 Preview Deployment Cleaned Up

The preview deployment for this PR has been removed.

Copy link

🧹 Preview Deployment Cleaned Up

The preview deployment for this PR has been removed:

  • ✅ API worker deleted
  • ✅ KV metadata removed
  • ✅ R2 objects deleted

Thank you for your contribution! 🎉

Copy link

github-actions bot commented Jul 10, 2025

🚀 Demo Preview Ready!

Preview URL: https://demo.rozich.net/ag-grid-react-components-pr-88/
API URL: https://demo.rozich.net/ag-grid-react-components-pr-88/api/

⚠️ Deployment tests failed - please check the workflow logs

This preview will be available for testing until the PR is merged or closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

has-preview status: merged PR merged to main branch

Projects

Status: 🚀 Merged

Development

Successfully merging this pull request may close these issues.

1 participant