Skip to content

Conversation

@pull
Copy link

@pull pull bot commented Sep 10, 2025

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.3)

Can you help keep this open source service alive? 💖 Please sponsor : )

charislam and others added 6 commits September 9, 2025 22:14
…with an issue (#38422)

* change the location of importing SpreadsheetImportPreview component

* Refactor error handling in SpreadsheetImport components

- show the row number with an issue to give users more actionable message
- show only the first error since Papaparse has an issue for row value in errors array

* Refactor error message for spreadsheet import alerts

* Remove items from `results.errors` with duplicate row and code values

* add comment

* Update error badge to show dynamic issue count

* Make errors stand out

* Refactor import/export for SpreadsheetImportPreview component

* Refactor import statements for SpreadsheetImport component

* use size-[value] class

* Refactor error handling to remove duplicates in preview

* Make only items with details clickable

* use proper list element

* use `button` element for clickable items

* add transition for transform

---------

Co-authored-by: Charis <[email protected]>
* new home top

* advisors

* fix ts

* add advisor section

* Update apps/studio/components/interfaces/Linter/Linter.utils.tsx

Co-authored-by: Charis <[email protected]>

* Update apps/studio/components/interfaces/Linter/LintDetail.tsx

Co-authored-by: Charis <[email protected]>

* Update apps/studio/components/interfaces/Linter/LinterDataGrid.tsx

Co-authored-by: Charis <[email protected]>

* Update packages/ui-patterns/src/Row/index.tsx

Co-authored-by: Charis <[email protected]>

* Update packages/ui-patterns/src/Row/index.tsx

Co-authored-by: Charis <[email protected]>

* Update apps/studio/components/interfaces/HomeNew/AdvisorSection.tsx

Co-authored-by: Charis <[email protected]>

* row refactor

---------

Co-authored-by: Charis <[email protected]>
Co-authored-by: Joshen Lim <[email protected]>
* Gracefully handle feature flag set up

* Update FeatureFlagProvider to support granular provider toggling, and make getConfigCatFlags an optional prop

* Nit

* Fix TS
@pull pull bot locked and limited conversation to collaborators Sep 10, 2025
@pull pull bot added the ⤵️ pull label Sep 10, 2025
@pull pull bot had a problem deploying to Studio E2E Tests September 10, 2025 04:13 Error
@pull pull bot merged commit 3963cbd into code:master Sep 10, 2025
@pull pull bot had a problem deploying to Studio E2E Tests September 10, 2025 04:13 Error
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants