Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 7 additions & 1 deletion .changeset/config.json
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,13 @@
}
],
"commit": false,
"fixed": [],
"fixed": [
[
"@wso2/oxygen-ui",
"@wso2/oxygen-ui-icons-react",
"@wso2/oxygen-ui-charts-react"
]
],
"linked": [],
"access": "public",
"baseBranch": "main",
Expand Down
61 changes: 61 additions & 0 deletions .github/ISSUE_TEMPLATE/bug.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,61 @@
name: "🐞 Report a Bug"
description: Create an issue if something does not work as expected.
labels: ["Type/Bug"]
body:
- type: textarea
id: background
attributes:
label: Description
description: Please share a clear and concise description of the problem.
placeholder: Description
- type: textarea
id: steps
attributes:
label: Steps to Reproduce
description: List the steps you followed when you encountered the issue.
validations:
required: true
- type: dropdown
id: area
attributes:
label: Please select the area the issue is related to
options:
- '@oxygen-ui/react'
- '@oxygen-ui/react-icons'
- '@oxygen-ui/react-charts'
- '@oxygen-ui/eslint-plugin'
- '@oxygen-ui/esbuild-plugin-inline-css-fonts'
- '@oxygen-ui/vite-plugin'
- 'oxygen-ui-docs'
- 'samples'
- 'other'
default: 1
multiple: true
validations:
required: true
- type: input
id: version
attributes:
label: Version
description: Enter package version or commit hash. (Use `N/A` if not applicable)
validations:
required: true
- type: textarea
id: environment
attributes:
label: Environment Details (with versions)
description: Mention the environment details (OS, Browser, React version, etc.) that the UI library is running on.
validations:
required: false
- type: checkboxes
id: reporter-checklist
attributes:
label: Reporter Checklist
description: Please ensure the following before submitting the issue.
options:
- label: "I have searched the existing issues and this is not a duplicate."
required: true
- label: "I have provided all the necessary information."
required: true
- label: "I have tested the issue on the latest version of the package."
required: true
55 changes: 55 additions & 0 deletions .github/ISSUE_TEMPLATE/improvement.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,55 @@
name: "🚀 Improvement Request"
description: Suggest an improvement for Oxygen UI.
labels: ["Type/Improvement"]
body:
- type: textarea
id: limitation
attributes:
label: Current Limitation
description: Describe the current limitation.
validations:
required: true
- type: textarea
id: suggestion
attributes:
label: Suggested Improvement
description: Describe the improvement you suggest.
validations:
required: true
- type: dropdown
id: area
attributes:
label: Please select the package issue is related to

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟡 Minor

Minor grammar fix for consistency.

The label is missing "the" before "issue". This should match the phrasing in bug.yml ("the issue is related to").

Proposed fix
-      label: Please select the package issue is related to
+      label: Please select the package the issue is related to
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
label: Please select the package issue is related to
label: Please select the package the issue is related to
🤖 Prompt for AI Agents
In @.github/ISSUE_TEMPLATE/improvement.yml at line 22, Update the label text
value that reads "Please select the package issue is related to" to include the
missing article so it matches bug.yml; change the string to "Please select the
package the issue is related to" (locate and edit the label key with that exact
text in .github/ISSUE_TEMPLATE/improvement.yml).

options:
- '@oxygen-ui/react'
- '@oxygen-ui/react-icons'
- '@oxygen-ui/react-charts'
- '@oxygen-ui/eslint-plugin'
- '@oxygen-ui/esbuild-plugin-inline-css-fonts'
- '@oxygen-ui/vite-plugin'
- 'oxygen-ui-docs'
- 'samples'
- 'other'
default: 1
multiple: true
validations:
required: true
- type: input
id: version
attributes:
label: Version
description: Enter package version or commit hash. (Use `N/A` if not applicable)
validations:
required: true
- type: checkboxes
id: reporter-checklist
attributes:
label: Reporter Checklist
description: Please review and confirm the following items before opening the improvement request.
options:
- label: "I have searched the existing issues and this is not a duplicate."
required: true
- label: "I have provided all the necessary information."
required: true
- label: "I have verified the improvement is not available in the latest version of the package."
required: true
62 changes: 62 additions & 0 deletions .github/ISSUE_TEMPLATE/new_feature.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,62 @@
name: "💡 New Feature Request"
description: Suggest new functionality and features for Oxygen UI.
labels: ["Type/NewFeature"]
body:
- type: textarea
id: problem
attributes:
label: Problem
description: What is the problem this feature will solve?
validations:
required: true
- type: textarea
id: solution
attributes:
label: Proposed Solution
description: Describe the solution you'd like to have.
validations:
required: true
- type: textarea
id: alternatives
attributes:
label: Alternatives
description: Describe any alternatives have you considered

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟡 Minor

Grammar fix needed.

The description has incorrect word order.

Proposed fix
-      description: Describe any alternatives have you considered
+      description: Describe any alternatives you have considered
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
description: Describe any alternatives have you considered
description: Describe any alternatives you have considered
🤖 Prompt for AI Agents
In @.github/ISSUE_TEMPLATE/new_feature.yml at line 23, The YAML description
string is grammatically incorrect; update the value for the description key (the
line currently reading "Describe any alternatives have you considered") to
correct word order — e.g., "Describe any alternatives you have considered" — so
the prompt reads naturally; locate the description entry in
.github/ISSUE_TEMPLATE/new_feature.yml and replace the text accordingly.

validations:
required: false
- type: dropdown
id: area
attributes:
label: Please select the package issue is related to

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟡 Minor

Minor grammar fix for consistency.

Same issue as in improvement.yml - missing "the" before "issue".

Proposed fix
-      label: Please select the package issue is related to
+      label: Please select the package the issue is related to
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
label: Please select the package issue is related to
label: Please select the package the issue is related to
🤖 Prompt for AI Agents
In @.github/ISSUE_TEMPLATE/new_feature.yml at line 29, Update the label string
"Please select the package issue is related to" by inserting the missing article
so it reads "Please select the package the issue is related to"; locate the
label key in the new_feature YAML template (the line currently containing that
exact label value) and change the text to include "the" for grammatical
consistency with improvement.yml.

options:
- '@oxygen-ui/react'
- '@oxygen-ui/react-icons'
- '@oxygen-ui/react-charts'
- '@oxygen-ui/eslint-plugin'
- '@oxygen-ui/esbuild-plugin-inline-css-fonts'
- '@oxygen-ui/vite-plugin'
- 'oxygen-ui-docs'
- 'samples'
- 'other'
default: 1
multiple: true
validations:
required: true
- type: input
id: version
attributes:
label: Version
description: Enter package version or commit hash. (Use `N/A` if not applicable)
validations:
required: true
- type: checkboxes
id: reporter-checklist
attributes:
label: Reporter Checklist
description: Please review and confirm the following items before opening the feature request.
options:
- label: "I have searched the existing issues and this is not a duplicate."
required: true
- label: "I have provided all the necessary information."
required: true
- label: "I have verified the feature on the latest version of the package."

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟡 Minor

Clarify the checklist item for new feature requests.

The current wording "I have verified the feature on the latest version" doesn't make sense for a new feature request—the feature doesn't exist yet. The intent appears to be confirming the feature isn't already available.

Proposed fix
-        - label: "I have verified the feature on the latest version of the package."
+        - label: "I have verified the feature is not available in the latest version of the package."
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
- label: "I have verified the feature on the latest version of the package."
- label: "I have verified the feature is not available in the latest version of the package."
🤖 Prompt for AI Agents
In @.github/ISSUE_TEMPLATE/new_feature.yml at line 61, Update the checklist
label text (currently "I have verified the feature on the latest version of the
package.") to clearly state that submitters checked the feature is not already
available; replace that label string with a phrasing like "I have confirmed this
feature is not available in the latest release" so the intent is unambiguous
when reviewing new feature requests.

required: true
24 changes: 24 additions & 0 deletions .github/pull_request_template.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,24 @@
### Purpose

<!-- Describe the problem, feature, improvement or the change introduced by the PR briefly. Add screenshots/GIFs if UI/UX changes are introduced. -->

### Related Issues

- N/A

### Related PRs

- N/A

### Checklist

- [ ] Followed the [CONTRIBUTING](https://github.com/wso2/oxygen-ui/blob/main/CONTRIBUTING.md) guidelines.
- [ ] Manual test round performed and verified.
- [ ] Documentation provided. (Add links if there are any)
- [ ] Unit tests provided. (Add links if there are any)
- [ ] Storybook stories updated/added (if applicable)

### Security checks

- [ ] Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines?

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟡 Minor

Fix the bare URL and use HTTPS.

The static analysis tool flagged a bare URL. Additionally:

  • The trailing ? appears to be a typo.
  • Use HTTPS instead of HTTP for security.
Proposed fix
-- [ ] Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines?
+- [ ] Followed secure coding standards in [WSO2 Secure Engineering Guidelines](https://wso2.com/technical-reports/wso2-secure-engineering-guidelines)
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
- [ ] Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines?
- [ ] Followed secure coding standards in [WSO2 Secure Engineering Guidelines](https://wso2.com/technical-reports/wso2-secure-engineering-guidelines)
🧰 Tools
🪛 markdownlint-cli2 (0.18.1)

23-23: Bare URL used

(MD034, no-bare-urls)

🤖 Prompt for AI Agents
In @.github/pull_request_template.md at line 23, Replace the bare HTTP URL in
the checklist item text "- [ ] Followed secure coding standards in
http://wso2.com/technical-reports/wso2-secure-engineering-guidelines?" with a
secure HTTPS link and remove the trailing question mark; e.g., change it to a
descriptive Markdown link like "- [ ] Followed secure coding standards in
https://wso2.com/technical-reports/wso2-secure-engineering-guidelines" (or use
"[WSO2 Secure Engineering
Guidelines](https://wso2.com/technical-reports/wso2-secure-engineering-guidelines)"
for clarity).

- [ ] Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets?
3 changes: 2 additions & 1 deletion packages/oxygen-ui-charts-react/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@
"recharts": "3.3.0"
},
"peerDependencies": {
"@wso2/oxygen-ui": "workspace:^",
"@wso2/oxygen-ui": ">=0.1.0",
"react": "catalog:",
"react-dom": "catalog:",
"react-is": "19.0.0"
Expand All @@ -53,6 +53,7 @@
"@types/react-dom": "catalog:",
"@typescript-eslint/eslint-plugin": "catalog:",
"@typescript-eslint/parser": "catalog:",
"@wso2/oxygen-ui": "workspace:^",
"esbuild": "catalog:",
"eslint": "catalog:",
"eslint-plugin-react": "catalog:",
Expand Down
3 changes: 2 additions & 1 deletion packages/oxygen-ui/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -74,11 +74,11 @@
"@mui/x-data-grid": "catalog:",
"@mui/x-date-pickers": "catalog:",
"@mui/x-tree-view": "catalog:",
"@wso2/oxygen-ui-icons-react": "workspace:^",
"date-fns": "4.1.0",
"prismjs": "^1.30.0"
},
"peerDependencies": {
"@wso2/oxygen-ui-icons-react": ">=0.1.0",
"react": "catalog:",
"react-dom": "catalog:"
},
Expand All @@ -91,6 +91,7 @@
"@typescript-eslint/parser": "catalog:",
"@vitest/ui": "catalog:",
"@wso2/esbuild-plugin-inline-css-fonts": "workspace:^",
"@wso2/oxygen-ui-icons-react": "workspace:^",
"esbuild": "catalog:",
"eslint": "catalog:",
"eslint-plugin-react": "catalog:",
Expand Down
12 changes: 6 additions & 6 deletions pnpm-lock.yaml

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading