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

feat: Add a script to call the massImport method #193

Merged
merged 1 commit into from
Nov 29, 2023

Conversation

alainncls
Copy link
Collaborator

@alainncls alainncls commented Sep 14, 2023

What does this PR do?

Adds a script to transform raw input data to attestable data
Adds a script able to batch call the massImport method

Related ticket

Fixes #192

Type of change

  • Chore
  • Bug fix
  • New feature
  • Documentation update

Check list

  • Unit tests for any smart contract change
  • Contracts and functions are documented

@alainncls alainncls self-assigned this Sep 14, 2023
@alainncls alainncls marked this pull request as draft September 14, 2023 10:34
@codecov-commenter
Copy link

codecov-commenter commented Sep 14, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (a9bf0f5) 94.24% compared to head (d5f21ad) 94.24%.
Report is 1 commits behind head on dev.

Additional details and impacted files
@@           Coverage Diff           @@
##              dev     #193   +/-   ##
=======================================
  Coverage   94.24%   94.24%           
=======================================
  Files          19       19           
  Lines         313      313           
  Branches       68       68           
=======================================
  Hits          295      295           
  Misses          9        9           
  Partials        9        9           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@alainncls alainncls force-pushed the feature/add-a-script-to-call-the-massimport-method branch 2 times, most recently from 6caf231 to c51d063 Compare September 19, 2023 09:48
@alainncls alainncls force-pushed the feature/add-a-script-to-call-the-massimport-method branch 4 times, most recently from 6bcf473 to acd3363 Compare September 28, 2023 21:43
@alainncls alainncls force-pushed the feature/add-a-script-to-call-the-massimport-method branch 4 times, most recently from d6d2e02 to 406df9c Compare October 4, 2023 13:18
@alainncls alainncls force-pushed the feature/add-a-script-to-call-the-massimport-method branch 5 times, most recently from e3b0d01 to 01dc1dd Compare October 16, 2023 13:55
@alainncls alainncls force-pushed the feature/add-a-script-to-call-the-massimport-method branch 3 times, most recently from 7992dcb to 30a8a6d Compare October 20, 2023 13:09
@alainncls alainncls force-pushed the feature/add-a-script-to-call-the-massimport-method branch from 30a8a6d to 232240b Compare October 23, 2023 10:04
@alainncls alainncls force-pushed the feature/add-a-script-to-call-the-massimport-method branch 4 times, most recently from 9455858 to b826821 Compare November 9, 2023 10:35
@alainncls alainncls force-pushed the feature/add-a-script-to-call-the-massimport-method branch 7 times, most recently from 56fd2da to 7a52af1 Compare November 17, 2023 14:25
@alainncls alainncls force-pushed the feature/add-a-script-to-call-the-massimport-method branch 12 times, most recently from 86ad68c to 3a7253b Compare November 29, 2023 10:13
@alainncls alainncls marked this pull request as ready for review November 29, 2023 10:14
@alainncls alainncls force-pushed the feature/add-a-script-to-call-the-massimport-method branch from 3a7253b to 300f6ee Compare November 29, 2023 10:14
@alainncls alainncls force-pushed the feature/add-a-script-to-call-the-massimport-method branch from 300f6ee to cd874e6 Compare November 29, 2023 15:20
@alainncls alainncls enabled auto-merge (squash) November 29, 2023 15:20
@alainncls alainncls merged commit 36e6df7 into dev Nov 29, 2023
8 checks passed
@alainncls alainncls deleted the feature/add-a-script-to-call-the-massimport-method branch November 29, 2023 15:23
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.

[US] As an Admin, I want to mass import attestations from a large source of data
5 participants