-
Notifications
You must be signed in to change notification settings - Fork 17
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 virtual third parties #772
Conversation
Pull Request Test Coverage Report for Build 10965795567Details
💛 - Coveralls |
…feat/add-virtual-third-parties
src/Item/Item.service.ts
Outdated
@@ -11,7 +11,7 @@ import { ItemCuration } from '../Curation/ItemCuration' | |||
import { Bridge } from '../ethereum/api/Bridge' | |||
import { collectionAPI } from '../ethereum/api/collection' | |||
import { peerAPI } from '../ethereum/api/peer' | |||
import { thirdPartyAPI } from '../ethereum/api/thirdParty' | |||
// import { thirdPartyAPI } from '../ethereum/api/thirdParty' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
// import { thirdPartyAPI } from '../ethereum/api/thirdParty' |
src/Item/Item.service.ts
Outdated
@@ -49,6 +49,7 @@ import { | |||
isTPItem, | |||
toDBItem, | |||
} from './utils' | |||
import { ThirdPartyService } from '../ThirdParty/ThirdParty.service' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could you move this import above?
operation: 'UPDATE', | ||
level: 'ROW', | ||
function: update_updated_at_column_function_name, | ||
condition: 'OLD.* IS DISTINCT FROM NEW.*', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
shouldn't the updated_at field be updated with each update? or should it meet a specific condition?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The updated_at
column will get updated only if any column in the row gets updated to something new. If it doesn't, we'll be saving a write operation by not updating the updated_at
.
src/ThirdParty/ThirdParty.service.ts
Outdated
: ([] as ThirdParty[]), | ||
]) | ||
const graphThirdParties = fragments.map(toThirdParty) | ||
console.log({ graphThirdParties, virtualThirdParties }) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
console.log({ graphThirdParties, virtualThirdParties }) |
…feat/add-virtual-third-parties
This PR adds the capability of using creating virtual third parties. That is, third parties created with the purpose of being temporary third parties until the registration is done on chain.