Skip to content

feat: move event category group edition to MUI#996

Open
priscila-moneo wants to merge 3 commits into
feature/move-event-category-groups-muifrom
feature/move-event-category-groups-edit-mui
Open

feat: move event category group edition to MUI#996
priscila-moneo wants to merge 3 commits into
feature/move-event-category-groups-muifrom
feature/move-event-category-groups-edit-mui

Conversation

@priscila-moneo

Copy link
Copy Markdown

@coderabbitai

coderabbitai Bot commented Jun 24, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 906ff8f0-ec77-4330-8bba-32a5cefa5ee1

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/move-event-category-groups-edit-mui

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

sx={{ mb: 1 }}
>
<Grid2 size="grow">
<Autocomplete

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

why not use uicore MuiFormikAsyncAutocomplete ? I think this should be its own component either way. Also @smarcet do we really need this input to be an async select ? Seems to me that tracks are never too many and a normal select would suffice.

sx={{ mb: 1 }}
>
<Grid2 size="grow">
<Autocomplete

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Same comment as for Track async select,. Should be at least its own component, but @smarcet pretty sure track groups cannot be more than a few dozens realistically

Comment thread src/pages/events/components/event-category-group-dialog.js
Comment thread src/pages/events/components/event-category-group-dialog.js Outdated
}
};

const MuiFormikColorInput = ({ name, ...props }) => {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

this should go in uicore

@priscila-moneo
priscila-moneo force-pushed the feature/move-event-category-groups-mui branch from 2d543dc to f9057ba Compare June 25, 2026 23:00
@smarcet
smarcet requested a review from Copilot July 14, 2026 14:32

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

priscila-moneo and others added 2 commits July 21, 2026 17:13
Signed-off-by: Priscila Moneo <priscila_moneo@hotmail.com.ar>
Signed-off-by: Tomás Castillo <tcastilloboireau@gmail.com>
@tomrndom
tomrndom force-pushed the feature/move-event-category-groups-edit-mui branch from c1a35ee to e399fe2 Compare July 21, 2026 20:14
Signed-off-by: Tomás Castillo <tcastilloboireau@gmail.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.

4 participants