Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Builtin Menu Reorganization #5709

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from
Open

Conversation

lykinsbd
Copy link
Contributor

No description provided.

@github-actions github-actions bot added the group/backend Issue related to the backend (API Server, Git Agent) label Feb 10, 2025
Copy link

codspeed-hq bot commented Feb 10, 2025

CodSpeed Performance Report

Merging #5709 will not alter performance

Comparing bdl-20250204-menu_reorg (487c901) with develop (cacbea1)

Summary

✅ 11 untouched benchmarks

@github-actions github-actions bot added the group/frontend Issue related to the frontend (React) label Feb 10, 2025
…l-20250204-menu_reorg

And link to IFC-1213
@lykinsbd
Copy link
Contributor Author

Have the changes functioning and tests fixed, need to look at migrations now.

@lykinsbd lykinsbd marked this pull request as ready for review February 12, 2025 18:30
@lykinsbd lykinsbd requested review from a team as code owners February 12, 2025 18:30
@ajtmccarty
Copy link
Contributor

is this waiting on the backend team to make a migration or is that going to be a separate PR?

@lykinsbd
Copy link
Contributor Author

I got the information I was looking for from @gmazoyer this morning; I am working on the migrations right now, moving this back to a draft for the interim as I work on that. If I'm unable to make progress today on that, I may ask someone from the backend to assist and push migrations to my branch.

@lykinsbd lykinsbd marked this pull request as draft February 13, 2025 17:23
@lykinsbd
Copy link
Contributor Author

Per conversation with Damien, merging this as-is into Develop. He will look at the migration required.

@lykinsbd lykinsbd marked this pull request as ready for review February 18, 2025 21:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
group/backend Issue related to the backend (API Server, Git Agent) group/frontend Issue related to the frontend (React)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants