Skip to content
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

Csv import fixes #1584

Merged
merged 22 commits into from
Feb 27, 2025
Merged

Csv import fixes #1584

merged 22 commits into from
Feb 27, 2025

Conversation

jbair06
Copy link
Member

@jbair06 jbair06 commented Feb 26, 2025

Description
The current templates in use for CSV importing were slightly different than those provided originally. As such, this PR adds the new features of the templates, in addition to fixing some visually issues when displaying the transfer data.

Related issue(s):

Fixes #1571, #1569

Signed-off-by: John Bair <[email protected]>
Signed-off-by: John Bair <[email protected]>
Signed-off-by: John Bair <[email protected]>
Signed-off-by: John Bair <[email protected]>
@jbair06 jbair06 requested review from SvetBorislavov, yiliev0 and a team as code owners February 26, 2025 18:54
Copy link

codecov bot commented Feb 26, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.10%. Comparing base (c7378db) to head (b171202).
Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1584   +/-   ##
=======================================
  Coverage   99.10%   99.10%           
=======================================
  Files         161      161           
  Lines        6148     6148           
  Branches     1138     1146    +8     
=======================================
  Hits         6093     6093           
+ Misses         55       52    -3     
- Partials        0        3    +3     

see 3 files with indirect coverage changes

Impacted file tree graph

@jsync-swirlds jsync-swirlds removed their request for review February 26, 2025 19:00
@SvetBorislavov SvetBorislavov merged commit ec6bb73 into main Feb 27, 2025
17 checks passed
@SvetBorislavov SvetBorislavov deleted the csv-import-fixes branch February 27, 2025 08:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CSV template can include a memo field per row
3 participants