Bug #11646
CSV import should not import blank rows
Status: | Verified | Start date: | 10/25/2017 | |
---|---|---|---|---|
Priority: | Medium | Due date: | ||
Assignee: | - | % Done: | 0% | |
Category: | CSV import | |||
Target version: | Release 2.5.0 | |||
Google Code Legacy ID: | Tested version: | 2.4 | ||
Sponsored: | No | Requires documentation: |
Description
Reported on the forum:
https://groups.google.com/forum/?utm_medium=email&utm_source=footer#!msg/ica-atom-users/oov5qBAIwDg/vwKjNjEDAwAJ
The CSV import should skip any rows consist entirely of a string of commas with no data, e.g:,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,
History
#1 Updated by Mike Cantelon over 4 years ago
- Status changed from New to Code Review
- Assignee set to Nick Wilkinson
PR for CR: https://github.com/artefactual/atom/pull/657
#2 Updated by Nick Wilkinson over 4 years ago
- Assignee changed from Nick Wilkinson to Steve Breker
Hi Steve, passing to you for CR.
#3 Updated by Steve Breker over 4 years ago
- Status changed from Code Review to Feedback
- Assignee changed from Steve Breker to Mike Cantelon
Looks good. Radda put a couple notes on the PR about capitalizing some of the new comments which is a good idea.
#4 Updated by Mike Cantelon over 4 years ago
- Status changed from Feedback to QA/Review
- Assignee changed from Mike Cantelon to Dan Gillean
Merged into qa/2.5.x.
#5 Updated by Dan Gillean over 4 years ago
- Target version set to Release 2.5.0
#6 Updated by Dan Gillean over 4 years ago
- Status changed from QA/Review to Verified
- Assignee deleted (
Dan Gillean)