-
Notifications
You must be signed in to change notification settings - Fork 16.6k
fix(dashboards): Use same decorators as FAB #36423
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
fix(dashboards): Use same decorators as FAB #36423
Conversation
|
CodeAnt AI is reviewing your PR. |
Code Review Agent Run #37bd28Actionable Suggestions - 0Review Details
Bito Usage GuideCommands Type the following command in the pull request comment and save the comment.
Refer to the documentation for additional commands. Configuration This repository uses Documentation & Help |
Nitpicks 🔍
|
|
CodeAnt AI finished reviewing your PR. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #36423 +/- ##
==========================================
+ Coverage 60.48% 67.97% +7.48%
==========================================
Files 1931 634 -1297
Lines 76236 46738 -29498
Branches 8568 5071 -3497
==========================================
- Hits 46114 31770 -14344
+ Misses 28017 13701 -14316
+ Partials 2105 1267 -838
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Vitor-Avila
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thanks for catching this
(cherry picked from commit 4a249a0)
(cherry picked from commit 4a249a0)
User description
SUMMARY
After #36246 the dashboard API was not setting the session cookie because we override the API but didn't include all the decorators from FAB. By including these, we can see the API sets the cookie correctly
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
Before

After

TESTING INSTRUCTIONS
Load the dashboard list page and make sure you see the cookie being set
ADDITIONAL INFORMATION
CodeAnt-AI Description
Dashboard list API sets session cookie and supports merged response options
What Changed
Impact
✅ Session cookie set for dashboard list✅ Dashboard list returns requested columns/labels/descriptions/order/title✅ Consistent list responses with other APIs💡 Usage Guide
Checking Your Pull Request
Every time you make a pull request, our system automatically looks through it. We check for security issues, mistakes in how you're setting up your infrastructure, and common code problems. We do this to make sure your changes are solid and won't cause any trouble later.
Talking to CodeAnt AI
Got a question or need a hand with something in your pull request? You can easily get in touch with CodeAnt AI right here. Just type the following in a comment on your pull request, and replace "Your question here" with whatever you want to ask:
This lets you have a chat with CodeAnt AI about your pull request, making it easier to understand and improve your code.
Example
Preserve Org Learnings with CodeAnt
You can record team preferences so CodeAnt AI applies them in future reviews. Reply directly to the specific CodeAnt AI suggestion (in the same thread) and replace "Your feedback here" with your input:
This helps CodeAnt AI learn and adapt to your team's coding style and standards.
Example
Retrigger review
Ask CodeAnt AI to review the PR again, by typing:
Check Your Repository Health
To analyze the health of your code repository, visit our dashboard at https://app.codeant.ai. This tool helps you identify potential issues and areas for improvement in your codebase, ensuring your repository maintains high standards of code health.