Skip to content

Add react-native-edge-glow library details#2661

Closed
aravi365 wants to merge 1 commit into
react-native-community:mainfrom
aravi365:patch-1
Closed

Add react-native-edge-glow library details#2661
aravi365 wants to merge 1 commit into
react-native-community:mainfrom
aravi365:patch-1

Conversation

@aravi365

Copy link
Copy Markdown

📝 Why & how

Adds a new library: react-native-edge-glow — a Siri/Gemini-style animated glow around the screen edges. It runs the animation on the UI thread via Reanimated, reads the device's real display corner radius on Android 12+ (via the public WindowInsets API), and ships with presets, a breathing pulse, and reduce-motion support. Dependencies are only react-native-svg + react-native-reanimated (no Skia).

✅ Checklist

  • Added library to react-native-libraries.json
  • Updated library in react-native-libraries.json

Added new library 'react-native-edge-glow' with images and architecture support.
@aravi365 aravi365 closed this Jul 17, 2026
@aravi365 aravi365 reopened this Jul 17, 2026
@aravi365 aravi365 closed this Jul 17, 2026
@aravi365 aravi365 reopened this Jul 17, 2026
@aravi365 aravi365 closed this Jul 17, 2026
@aravi365 aravi365 reopened this Jul 17, 2026
@aravi365

aravi365 commented Jul 17, 2026

Copy link
Copy Markdown
Author

The data-test-and-validate check is failing on a GitHub API error, not on my entry:

FORBIDDEN — path: ["repository", "stargazers"] — "Resource not accessible by integration"

My repo is public and reachable unauthenticated (stars, raw content, and the npm downloads API all resolve), so this looks like the fork-PR github.token (scoped contents: read) being unable to read repository.stargazers via GraphQL — a CI-token limitation rather than a problem with the submission.

Could a maintainer re-run in a trusted context or merge? My understanding is the authoritative data fetch runs post-merge with a privileged token, so the stars/metadata should populate correctly then. Happy to adjust the entry if anything's actually wrong with it. Thanks!
cc: @Simek

@Simek

Simek commented Jul 17, 2026

Copy link
Copy Markdown
Member

Hey @aravi365, it was failing initially on the lack data in npm API. It usually takes up to 24h for npm API after initial package publish to collect and return the required data.

I will re-run the workflow, by not sure if this would help, let's see, but you might need to recreate the PR. Also, in the current state lint checks will fail on whitespace. And one more thing, please move the new package at the end of the list - we use this order for "Recently added" sort option, see:

@Simek

Simek commented Jul 17, 2026

Copy link
Copy Markdown
Member

Yeah, re-run does not help (https://github.com/react-native-community/directory/actions/runs/29586792480), the run context have been compromised already. Please re-create the PR.

@Simek Simek closed this Jul 17, 2026
@aravi365 aravi365 mentioned this pull request Jul 17, 2026
1 task
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