forked from OpenStackweb/summit-admin
-
Notifications
You must be signed in to change notification settings - Fork 4
feat(sponsor-reports): port Sponsor Reports FE into Summit Admin (Sponsors → Reports) #997
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
Open
caseylocker
wants to merge
49
commits into
master
Choose a base branch
from
feature/sponsor-reports-port
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
49 commits
Select commit
Hold shift + click to select a range
d07a78a
feat(sponsor-reports): scaffold admin-gated Reports section under Spo…
caseylocker f7d0281
feat(sponsor-reports): port actions/reducers/utils + store wiring
caseylocker cdfebcc
feat(sponsor-reports): port shared report components (T.translate, ge…
caseylocker c2096a3
feat(sponsor-reports): Purchase Details report (MuiTable, conditional…
caseylocker 71cfdc1
test(sponsor-reports): cover CSV+pagination/sort dispatch; drop vesti…
caseylocker c3b4a02
feat(sponsor-reports): Sponsor Assets report + drill-down (rewired ro…
caseylocker 084589c
feat(sponsor-reports): Reports landing + integration
caseylocker db5680f
chore(sponsor-reports): final-review cleanup — drilldown errors, i18n…
caseylocker bf38fbf
refactor(sponsor-reports): drop vestigial export-disabled; route to R…
caseylocker 07c3261
fix(sponsor-reports): UI polish (fonts, breadcrumb trail, asset spacing)
caseylocker e7357b5
fix(sponsor-reports): address CodeRabbit review (PR #997)
caseylocker 7c4eaa9
fix(sponsor-reports): address Copilot review (PR #997)
caseylocker d342f26
chore(sponsor-reports): request dedicated sponsor-reports scopes (rev…
caseylocker 18851e7
feat(sponsor-reports): add purchase-details lines report action
caseylocker 0e37fe6
feat(sponsor-reports): add purchase-details lines reducer + store reg…
caseylocker dd5e8a2
feat(sponsor-reports): add bucketLinesBySponsor grouping util
caseylocker 2666e27
feat(sponsor-reports): add LinesManifestView collapsible per-line man…
caseylocker 4667f8b
feat(sponsor-reports): add Orders/Line-Items report view toggle
caseylocker 17d4ff6
feat(sponsor-reports): wire Line Items manifest view into Purchase De…
caseylocker c1cc057
style(sponsor-reports): strip trailing whitespace in LinesManifestView
caseylocker e06b1e9
refactor(sponsor-reports): address PR #997 convention review (smarcet)
caseylocker 3315801
test(sponsor-reports): add lines slice to layout test store (fix CI)
caseylocker 562f87e
fix(sponsor-reports): wrap overflowing asset filenames in drilldown l…
caseylocker f2cac7e
fix(sponsor-reports): readable toggle + summary-tile typography
caseylocker 645fe0c
feat(sponsor-reports): visible sponsor avatars + white header-icon theme
caseylocker 809356f
feat(sponsor-reports): enable CSV export on the Line Items view
caseylocker fd1296c
refactor(sponsor-reports): add isPositiveIntId + htmlToPlainText to m…
caseylocker b22dc41
refactor(sponsor-reports): co-locate report-query + add shared purcha…
caseylocker 500cab9
refactor(sponsor-reports): use methods.htmlToPlainText, drop reports-…
caseylocker a0310a1
refactor(sponsor-reports): purchase-details CSV export via action thunks
caseylocker 306ce26
refactor(sponsor-reports): sponsor-asset CSV export via action thunks…
caseylocker 6757a71
refactor(sponsor-reports): delete reports-api util; wire isPositiveIn…
caseylocker f728156
refactor(sponsor-reports): co-locate report-errors + reports-money ou…
caseylocker a798b4e
fix(sponsor-reports): update constants import path in moved report-er…
caseylocker 362efb1
Merge origin/master into feature/sponsor-reports-fe-conventions (sync…
caseylocker a18a6d5
refactor(reports): replace formatUsd with uicore currencyAmountFromCents
caseylocker 9a614a1
test(reports): use cents fixtures in purchase-details reducer test
caseylocker 473c625
refactor(reports): dedup per-page/ms constants via src/utils/constants
caseylocker 35974ee
fix(sponsor-reports): return Promise.resolve() on all thunk guard bra…
caseylocker 6dcd257
refactor(sponsor-reports): fold statusTone into StatusPill
caseylocker 78bc8c6
refactor(reports): replace hand-rolled formatCheckoutTime with moment
caseylocker 7632cbe
refactor(sponsor-reports): adopt useSnackbarMessage hook for 412 vali…
caseylocker 57a1400
test(sponsor-reports): mock useSnackbarMessage in layout test
caseylocker 06e65f0
refactor(T6): move manifest-grouping into reports feature dir
caseylocker b533901
refactor(sponsor-reports): T7 — dedup page-size constant + fix defaul…
caseylocker dc18ea4
refactor(sponsor-reports): convert nextDayStartIso to moment-timezone
caseylocker aceafed
test(OrdersTable): pin moment.utc offset+malformed contract (T9)
caseylocker 13ec334
refactor(sponsor-reports): use repo-native isImageUrl + DEFAULT_CURRE…
caseylocker 7eff215
Merge remote-tracking branch 'origin/master' into feature/sponsor-rep…
caseylocker File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.