Skip to content

[FIX] Add source file for scatterchart#395

Merged
jgbernalp merged 1 commit intoperses:mainfrom
lachieh:lachieh/fix-scatterchart-entry
Sep 30, 2025
Merged

[FIX] Add source file for scatterchart#395
jgbernalp merged 1 commit intoperses:mainfrom
lachieh:lachieh/fix-scatterchart-entry

Conversation

@lachieh
Copy link
Copy Markdown
Contributor

@lachieh lachieh commented Sep 30, 2025

Description

When working on #391, I noticed that the ScatterChart plugin output was not consistent with other plugins. This change updates the rsbuild.config.ts to align with the standard entry point used in other plugins.

Checklist

  • Pull request has a descriptive title and context useful to a reviewer.
  • Pull request title follows the [<catalog_entry>] <commit message> naming convention using one of the
    following catalog_entry values: FEATURE, ENHANCEMENT, BUGFIX, BREAKINGCHANGE, DOC,IGNORE.
  • All commits have DCO signoffs.

UI Changes

  • Changes that impact the UI include screenshots and/or screencasts of the relevant changes.
  • Code follows the UI guidelines.

@jgbernalp
Copy link
Copy Markdown
Contributor

@lachieh mind adding your DCO?

Signed-off-by: Lachlan Heywood <lachieh@users.noreply.github.com>
@lachieh lachieh force-pushed the lachieh/fix-scatterchart-entry branch from 97813fe to 3fe6faa Compare September 30, 2025 16:29
@lachieh
Copy link
Copy Markdown
Contributor Author

lachieh commented Sep 30, 2025

Yep, apologies, that's done!

@jgbernalp jgbernalp added this pull request to the merge queue Sep 30, 2025
Merged via the queue into perses:main with commit aa04b46 Sep 30, 2025
13 checks passed
@lachieh lachieh deleted the lachieh/fix-scatterchart-entry branch September 30, 2025 16:46
abelyakin pushed a commit to abelyakin/plugins that referenced this pull request Jan 14, 2026
Signed-off-by: Lachlan Heywood <lachieh@users.noreply.github.com>
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