fix: keep the tab order

[US579]

SUMMARY

When export all tabs to a PDF in report & alert, user noticed that the all tabs export doesn't seem to export the tabs in any specific order (our report with 3 tabs exported with tab 2, then 3, then 1 in the PDF)

reference: #30694 (comment)

Context

The exported tabs are now ordered based on the sequence of anchors under the all_tabs field from the tabs endpoint response. This enhancement aligns the export order with the order displayed on the dashboard.

BEFORE PDF

before.pdf

AFTER PDF

after.pdf

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

• Has associated issue:
• Required feature flags:
• Changes UI
• Includes DB Migration (follow approval process in SIP-59)
  • Migration is atomic, supports rollback & is backwards-compatible
  • Confirm DB migration upgrade and downgrade tested
  • Runtime estimates and downtime expectations provided
• Introduces new feature or API
• Removes existing feature or API

[US579]

FYI @yousoph

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.77%. Comparing base (d5a4815) to head (5f1fb73).

Additional details and impacted files

@@             Coverage Diff             @@
##           master   #30888       +/-   ##
===========================================
+ Coverage        0   83.77%   +83.77%     
===========================================
  Files           0      538      +538     
  Lines           0    39152    +39152     
===========================================
+ Hits            0    32799    +32799     
- Misses          0     6353     +6353     

Flag	Coverage Δ
hive	48.75% <12.50%> (?)
mysql	76.51% <100.00%> (?)
postgres	76.56% <100.00%> (?)
presto	53.26% <12.50%> (?)
python	83.77% <100.00%> (?)
sqlite	76.03% <100.00%> (?)
unit	60.89% <12.50%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[rusackas]

LGTM, but @eschutho might be better poised to actually test this before approving/merging.

[eschutho]

@US579 was the self.response not including the correct mimetype? I would think that all the responses should be json, but maybe @dpgaspar has more info on this.

[US579]

yep it does has mimetype, but jsonify func will reorder the dict, and it is part of the func in self.response

This is a test example

with response