Development #11
Merged
Development #11
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Created permissions api package for handling webapis permissions
Add documentation comments and readme, resolved pr comments
Add New Permissions API Module
- Added new permissions to the PermissionsAPI class. - Enhanced the HTML interface for displaying permission states. - Updated the Nx configuration in nx.json for better input handling. - Introduced a GitHub Actions workflow for CI verification. - Adjusted .gitignore to exclude additional files and directories. - Refined ESLint configuration to ignore dist directory.
…ssionsAPI - Removed the old index.html file and replaced it with a new demo/index.html for better structure and usability. - Introduced demo/index.js to handle PermissionsAPI interactions and improve user experience. - Updated PermissionsAPI.ts to enhance type definitions and improve handler functionality. - Refactored types.ts to align with new handler implementations.
Enhancements and Refactoring of PermissionsAPI with CI/CD Optimization
- Added error handling in PermissionsAPI to throw an error if the handler is closed. - Updated type definitions in types.ts to document the new error throwing behavior. - Removed the nxCloudAccessToken from nx.json for security reasons.
- Changed package manager version from pnpm@9.2.0 to pnpm@9 for consistency.
- Added environment variable NX_CLOUD_ACCESS_TOKEN to the verify.yml workflow for improved CI configuration.
…N in job environment - Moved the NX_CLOUD_ACCESS_TOKEN environment variable to the job level in verify.yml for improved clarity and organization.
…_TOKEN usage - Simplified the command in the verify.yml workflow to echo the NX_CLOUD_ACCESS_TOKEN before running the Nx Cloud command, enhancing clarity and maintainability.
- Replaced `pnpm exec` with `pnpx` for running Nx affected commands in the verify.yml workflow, ensuring consistency and improved command execution.
…installation - Changed the pnpm install command in verify.yml from --frozen-lockfile to --no-frozen-lockfile to enable more flexible dependency resolution during CI runs.
…sionsAPI - Added '**/*.html' to the ESLint ignore patterns to exclude HTML files from linting. - Updated type reference in demo/index.js from PermissionsAPI.getPermissionHandler to getPermissionHandler for improved clarity.
Improve PermissionsAPI Error Handling and Remove nxCloud Access Token
…e permission handling
chore: ✨ Refactor PermissionsAPI to extend PermissionsCore and improve permission handling
chore: ✨ Add PR template and update PR instructions 📝
- Added gulp tooling for compiling distributions and demos\n- Add tsup configuration file for conditional configs and minification\n- Updated `package.json` scripts
feat: ✨ Add tsup configuration for building with minification options
feat: add demos directory to pnpm workspace configuration
|
| Command | Status | Duration | Result |
|---|---|---|---|
nx build @arcstack/demos |
✅ Succeeded | 9s | View ↗ |
nx demos @arcstack/webapis |
✅ Succeeded | 10s | View ↗ |
☁️ Nx Cloud last updated this comment at 2026-01-02 23:13:08 UTC
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.