[COMPASS-6543] Surface type warnings from parsing a CSV in the import modal Created: 22/Feb/23  Updated: 24/Apr/23  Resolved: 24/Apr/23

Status: Closed
Project: Compass
Component/s: Import/Export
Affects Version/s: None
Fix Version/s: 1.37.0

Type: Task Priority: Major - P3
Reporter: Rhys Howell Assignee: Le Roux Bodenstein
Resolution: Done Votes: 0
Labels: import, milestone-5
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Depends
depends on COMPASS-6423 analyzeCSVFields() import function an... Closed
depends on COMPASS-6536 Start using `analyzeCSVFields`, add s... Closed
Documented
Epic Link: COMPASS-5576
Story Points: 2
Documentation Changes: Needed
Documentation Changes Summary:

We should probably document this. I think Jocelyn Mendez will be in our next project sync to discuss.

Sprint: Iteration Whale, Iteration Zebrafish

 Description   

The warnings indicate potential errors that will occur when importing. These are shown when a user changes the type to something we don't auto detect.

We show the warnings with a banner at the bottom of the import modal. Let's show the value, type, and issue. We don't show a warning in the table like designs as those cells are only the first 10 columns and it's likely that the warning falls outside of those rows.
Let's use the `WarningSummary` component from `compass-components`. The warning banner is dismissible.

Designs:
https://www.figma.com/file/4sXj42XyicBsdQkYIugHaV/Compass-Import%2FExport?node-id=1304%3A48196&t=waeIls8QO2JWTpSd-1 



 Comments   
Comment by Githook User [ 24/Apr/23 ]

Author:

{'name': 'Le Roux Bodenstein', 'email': 'lerouxb@gmail.com', 'username': 'lerouxb'}

Message: feat(import): Surface type warnings COMPASS-6543 (#4273)

  • add export modal
  • code cleanup, remove old comments
  • Surface CSV field type warnings
  • tweak comment
  • comment tweak
  • update expected results

---------

Co-authored-by: Rhys Howell <rhysh@live.com>
Branch: COMPASS-6530-auto-add-curly-braces
https://github.com/mongodb-js/compass/commit/a415820b7fc3a42475872ed8e8c667417d504d9b

Comment by Githook User [ 22/Apr/23 ]

Author:

{'name': 'Le Roux Bodenstein', 'email': 'lerouxb@gmail.com', 'username': 'lerouxb'}

Message: feat(import): Surface type warnings COMPASS-6543 (#4273)

  • add export modal
  • code cleanup, remove old comments
  • Surface CSV field type warnings
  • tweak comment
  • comment tweak
  • update expected results

---------

Co-authored-by: Rhys Howell <rhysh@live.com>
Branch: main
https://github.com/mongodb-js/compass/commit/a415820b7fc3a42475872ed8e8c667417d504d9b

Generated at Wed Feb 07 22:43:31 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.