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.

Sign up for GitHub

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

Jump to bottom

Smart Contract listing endpoints and GraphQl duration metrics #97

Merged
82 commits merged into from
Sep 11, 2023
Merged

Smart Contract listing endpoints and GraphQl duration metrics #97

82 commits merged into from
Sep 11, 2023

Conversation

ghost
Copy link

@ghost ghost commented Sep 1, 2023 •
edited by ghost
Loading

Purpose

Task
https://concordium.atlassian.net/browse/CBW-1261

Changes

  • Adding pagination endpoint to list Smart Contracts and added and added duration metrics to GraphQl endpoints.
  • Adding missing events needed at contract to contract transfer.
  • Added Block Slot Time to all events generated such that event time can easily be shown.

Checklist

  • My code follows the style of this project.
  • [] The code compiles without warnings.
  • I have performed a self-review of the changes.
  • I have documented my code, in particular the intent of the
    hard-to-understand areas.
  • (If necessary) I have updated the CHANGELOG.

schwartz-concordium added 30 commits August 17, 2023 15:37
Made entities and import flow
d5e8afa
Added test to aggregate which validates events are stored
cdf8265
Merge branch 'main' into cbw-1248/implementation-smart-contract-aggre…
bc505e9 
…gate
Added test for other read cases
fbf296d
Added smart contract entity
aa922e0
Added Smart Contract Aggregate to Startup
81b34ab
Added import job
58156e1
Finished repository test
dcb5256
Fixed aggregate test
ca0522a
Added data source
b73da60
Added awaits on jobs on node import
91b0eaf
Added contract version to contract initialize and update events
fc47cb3
Added tests which validates uniqueness contrains
50f6167
Remove todos and added simple resilience in import loop
293329a
Fixed dependency injection issue
1cfb11c
Fix issues when testing job
aa91b17
Small updates
d575289
Updated changelog
f63bf17
Moved classes to own files
ef4550d
Added logic to Smart Contract Repository which can fetch batch
eae5cb1
Made batch import instead of processing single rows
bb162ca
Clean up unused code
952e48b
Fix limit on smart contract database job
4a40b00
Remove unnecessary batch
e708617
Added duration- and height metrics
40e474d
Added health states to application
d565919
Fixed health output
b3cc30b
Adding temp CI
5411768
Updating change log
ea18072
Remove some debugging and added nugets
75f2851
schwartz-concordium added 18 commits September 5, 2023 09:15
Corrected naming
bff53f4
Made repository for contract jobs
5bf501e
Migrated to options for feature flags
5843bdc
Moved interfaces to same file as classes where only one implementatio…
37b979e 
…n is expected.
Changed to date time offset
e9bc1c1
Added warnings not to change job identifier
152c486
Update range function with comments
3c682d0
Follow standards from EF regarding CS8618 compiler warning for C# 10
6769b70 
https://learn.microsoft.com/en-us/ef/core/miscellaneous/nullable-reference-types#non-nullable-properties-and-initialization
Refactored Contract Aggregate Node Import job to be more readable
3e7126e
Merge branch 'cbw-1248/implementation-smart-contract-aggregate' into …
8048a49 
…cbw-1249/add-observability
Resolve comments after merge
a30fb7c
Merge branch 'main' into cbw-1249/add-observability
9f028f2
Resolve issues after merge
05dfeb9
Merge branch 'cbw-1249/add-observability' into cbw-1261/sc-listing-en…
0c43cc3 
…dpoints
Resolve issues after merge
77ec045
Resolve comments
e26cb54
Minor refactoring after final review
de8e391
Merge branch 'cbw-1249/add-observability' into cbw-1261/sc-listing-en…
d47a38c 
…dpoints
Base automatically changed from cbw-1249/add-observability to main September 5, 2023 11:53
@ghost ghost requested a review from ali-concordium September 5, 2023 12:47
@ghost ghost merged commit 4150a04 into main Sep 11, 2023
1 check passed
@ghost ghost deleted the cbw-1261/sc-listing-endpoints branch September 11, 2023 07:33
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@ali-concordium