chore: sync all unraid-component files from api repo build#2111
chore: sync all unraid-component files from api repo build#2111zackspear wants to merge 4 commits into
Conversation
WalkthroughThis pull request removes the legacy manifest file from the unraid-components module, updates an existing manifest with revised client file references and a new timestamp, and introduces two new files in the UI directory. The new files include a CSS file with utility and responsive styles, and a JSON manifest mapping source files to their compiled outputs. These changes reflect a reorganization of asset tracking and improved management of UI component configurations. Changes
Possibly related PRs
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
emhttp/plugins/dynamix.my.servers/unraid-components/uui/ui.CaW9jS_X.css (1)
1-2: New CSS File – Comprehensive Utility & Responsive Styles
This entirely new CSS file introduces a robust set of utility classes and responsive styles, supporting modern UI frameworks with utility-first principles. Note that static analysis hints flagged an "unexpected value or character" at the start and duplicate font names (e.g., potential duplicate entries for "Emoji" and "Segoe"). Please verify whether these warnings are false positives or if adjustments are needed to remove redundancies and ensure compliance with CSS best practices.🧰 Tools
🪛 Biome (1.9.4)
[error] 1-1: Unexpected value or character.
Expected one of:
(parse)
[error] 1-1: Duplicate font names are redundant and unnecessary: Emoji
Remove duplicate font names within the property
(lint/suspicious/noDuplicateFontNames)
[error] 1-1: Duplicate font names are redundant and unnecessary: Segoe
Remove duplicate font names within the property
(lint/suspicious/noDuplicateFontNames)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
emhttp/plugins/dynamix.my.servers/unraid-components/uui/register._OziqIBF.js.mapis excluded by!**/*.map
📒 Files selected for processing (4)
emhttp/plugins/dynamix.my.servers/unraid-components/manifest.json(0 hunks)emhttp/plugins/dynamix.my.servers/unraid-components/nuxt/manifest.json(1 hunks)emhttp/plugins/dynamix.my.servers/unraid-components/uui/ui.CaW9jS_X.css(1 hunks)emhttp/plugins/dynamix.my.servers/unraid-components/uui/ui.manifest.json(1 hunks)
💤 Files with no reviewable changes (1)
- emhttp/plugins/dynamix.my.servers/unraid-components/manifest.json
🧰 Additional context used
🪛 Biome (1.9.4)
emhttp/plugins/dynamix.my.servers/unraid-components/uui/ui.CaW9jS_X.css
[error] 1-1: Unexpected value or character.
Expected one of:
(parse)
[error] 1-1: Duplicate font names are redundant and unnecessary: Emoji
Remove duplicate font names within the property
(lint/suspicious/noDuplicateFontNames)
[error] 1-1: Duplicate font names are redundant and unnecessary: Segoe
Remove duplicate font names within the property
(lint/suspicious/noDuplicateFontNames)
🔇 Additional comments (3)
emhttp/plugins/dynamix.my.servers/unraid-components/nuxt/manifest.json (2)
3-3: Updated Component File Name
The"file"property for theunraid-components.cliententry has been updated from_nuxt/unraid-components.client-BXUDi12i.jsto_nuxt/unraid-components.client-BDhuIo_x.js. Please ensure that all references to this file in the broader application (or dependency cache) have been updated accordingly.
11-11: Updated Timestamp
The"ts"property has been updated to1743633846, which should now reflect the current build output’s timestamp. Verify that this change aligns with the intended caching and asset invalidation strategies in your deployment process.emhttp/plugins/dynamix.my.servers/unraid-components/uui/ui.manifest.json (1)
1-12: New UI Manifest File Structure
This new JSON manifest cleanly maps source files to their compiled outputs. The entry for"src/register.ts"specifies an entry point with"isEntry": trueand producesregister._OziqIBF.js, while the"style.css"entry correctly links to the new CSS fileui.CaW9jS_X.css. The structure is clear and aligns with the updated organizational strategy.
…ternal callback library
|
found a regression. closing for now. |
|
please find the new PR here #2113 |
Summary by CodeRabbit
New Features
Updates
Chores