Skip to content

feat: Import/Export Contacts#127

Merged
fmorency merged 14 commits intoliftedinit:mainfrom
stanleyjones:import-export-contacts
Aug 3, 2023
Merged

feat: Import/Export Contacts#127
fmorency merged 14 commits intoliftedinit:mainfrom
stanleyjones:import-export-contacts

Conversation

@stanleyjones
Copy link
Copy Markdown
Contributor

@stanleyjones stanleyjones commented Jul 27, 2023

SCREENSHOT

Screenshot 2023-07-27 at 4 26 47 PM

Working but no error handling or anything yet.

Closes #123 .

@netlify
Copy link
Copy Markdown

netlify Bot commented Jul 27, 2023

Deploy Preview for lifted-alberto ready!

Name Link
🔨 Latest commit 220f897
🔍 Latest deploy log https://app.netlify.com/sites/lifted-alberto/deploys/64cbdfb82abb9900084151e2
😎 Deploy Preview https://deploy-preview-127--lifted-alberto.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@stanleyjones stanleyjones self-assigned this Jul 27, 2023
Copy link
Copy Markdown
Contributor Author

@stanleyjones stanleyjones left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some thoughts...

Comment thread src/features/contacts/components/contacts-management/contacts-management.tsx Outdated
Copy link
Copy Markdown
Contributor

@fmorency fmorency left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good! Can you add error handling/tests to this PR as well?
Thanks!

@codecov
Copy link
Copy Markdown

codecov Bot commented Aug 3, 2023

Codecov Report

Merging #127 (220f897) into main (8463881) will decrease coverage by 0.49%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #127      +/-   ##
==========================================
- Coverage   34.33%   33.84%   -0.49%     
==========================================
  Files          88       88              
  Lines        2234     2266      +32     
  Branches      648      656       +8     
==========================================
  Hits          767      767              
- Misses       1456     1488      +32     
  Partials       11       11              
Files Changed Coverage Δ
...onents/contacts-management/contacts-management.tsx 0.00% <0.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@stanleyjones stanleyjones requested a review from fmorency August 3, 2023 17:11
@stanleyjones
Copy link
Copy Markdown
Contributor Author

SUCCESS

Screenshot 2023-08-03 at 10 04 05 AM

WARNING

Screenshot 2023-08-03 at 10 04 20 AM

Copy link
Copy Markdown
Contributor

@fmorency fmorency left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@fmorency fmorency merged commit 920ecf6 into liftedinit:main Aug 3, 2023
stanleyjones added a commit to stanleyjones/alberto that referenced this pull request Sep 7, 2023
@stanleyjones stanleyjones deleted the import-export-contacts branch December 6, 2023 21:20
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.

Export/Import Contacts

2 participants