Reintroduce SimplePolicyInfo struct
#68
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
struct without the arrays is needed for web3j compatibility
|
Task linked: CU-865ceu9k6 Premium payment syncing fails |
amarinkovic
changed the title
SimplePolicyInfo struct [Cu-865ceu9k6]SimplePolicyInfo struct
Merged
tgeorgas
added a commit
that referenced
this pull request
Oct 2, 2023
* Token symbol uniqueness (#57) * feat: ensure token symbol is unique * fix: some typos and comments * fix: entity token uniqueness mapping and add test * fix: include external tokens in uniqueness check * test: LibERC token symbol * feat: update entity token info * doc: update generated markdown * test: entity token info update * Revert "doc: update generated markdown" This reverts commit 96ed7ee. # Conflicts: # docs/facets/IEntityFacet.md * fix: linter issues * test: failed ERC20 calls * refactor: fix solidity parser in prettier * ci: fix branch scopes * feat: add support for sepolia * feat: run anvil in docker * chore: remove a CI step not needed anymore * ci: reformat test workflow summary * ci: reformat test workflow summary * fix: sepolia upgrade hash make target * Update local node scripts (#64) * chore: ignore slither vscode files * feat: scripted scheduling * fix: update anvil bootstrap script * refactor: deployment on anvil, makefile * fix: makefile defaults and addresses * fix: anvil bootstrap * fix: anvil upgrade targets * chore: add sepolia broadcasts * chore: update sepolia broadcasts --------- Co-authored-by: Kevin Park <[email protected]> * chore: add make commands to initialize and upgrade a new diamond on anvil * refactor: align boostrap with new deployment scripts * fix: prep upgrade script method ref * feat: script anvil upgrade flow * chore: update gitignore for mnemonic files * ci: fix upgrade hash report * chore: remove stale goerli broadcasts * fix: upgrade stack too deep error * refactor: add some makefile targets for sepolia * fix: write the array of selectors to the same memory location to avoid stack too deep * chore: add deploy-contract-sepolia command, commit broadcast files * chore: minor makefile arg fix * chore: sepolia upgrade broadcasts * refactor(prep-upgrade): pull out generate function * chore: update prettier config * ci: don't generate upgrade hashes * chore: bump to v3.5.0 * feat: .env format proposal * feat: .env format proposal * chore: add sepolia broadcasts, acl facet updated * fix: add back simple policy info struct (#68) struct without the arrays is needed for web3j compatibility * Hotfix execute limit offer (#67) * fix(erc20-wrapper): allowance function modifiers * doc: update interface reference * fix: remove duplicate target * feat: add forking makefile targets * fix: deploy-mainnet-fork permissions * chore: upgrade sepolia diamond to v3.5.1 * ci: fix coverage env vars * fix: mainnet fork target for owner balance * Emit dividend withdrawn event (#59) * Deployer adapter script (#70) * feat: initial deployer script adapter * feat: script upgrade procedure * feat(deployer): add fork support * refactor: improve logging * fix: mainnet fork support * feat: add dry run option * refactor: improve flow and arguments for mainnet operation * refactor: better logging and argument handling * Enhance Fee and Commission Payment Routing for Marketplace and Policies (#66) * chore(deployer): fix mainnet fork upgrade flow * Refactor fees (#72) * refactor(fees): update fee schedule struct and mapping (WiP) * fix: compilation errors after refactoring * fix: trading fee payment * fix: remove non-default fee schedule * refactor: make getSigner pure * refactor: pass fee config externally as arrays instead of struct * chore: cleanup left over comments * doc: add replaceMakerBP natspec * fix: unique id for default fee schedule regardless of type * refactor: generic fee paid event * test: dynamic array size for comparing fee schedules * chore: fix formatting --------- Co-authored-by: Aleksandar Marinković <[email protected]> * chore: deployer script output formatting * chore: upgrade sepolia to v3.6.0 * chore(verification-util): pull out network into vars * rerfactor: generic fee schedule getter * chore: bump package version to v3.6.1 * doc: fix natspec * chore: broadcasts for v3.6.1 * chore: remove hotfix test this test is no longer relevant * fix: add view modifier to isObjectTokenizable * test(fees): trading fees coverage * test(fees): fix variable name typo * feat(fees): add zero premium fee support for platform * fix premium fee tests to consider stakeholder fees * chore: clean up pre policy setup in tests; clean up unused foundry props * chore: fix linter * chore: add platform flag for running docker on ARM CPUs * Calculate the trading fee on matching offer(s) schedule (#74) Co-authored-by: Kevin Park <[email protected]> * Fork Sanity Tests for Contract Upgrades (#76) * fix: calculate fee allocations breakdown * fix: setup default fee schedules inthe init diamond * fix: reintroduce order matching rounding parameter * fix: dividend withdraw fuzz test * chore: 4x the number of max rejects for fuzzing * Update access control part 1 (#77) * Withdraw restriction (#78) * fix: add contract verification to deployer script * chore: ignore macos files * chore: upgrade sepolia to v3.6.2 * chore: update email in package.json * Roles part 1 refinement (#79) * chore: upgrade sepolia to v3.6.3 smart contracts * refactor: deployer only to verify when not forking * fix: deployer account resolution * doc: deployer script explainer * Prevent role demotion (IM-24357) (#80) * test: hRoles - pretty print roles * feat: update SmartDeploy to pass in init diamond address * feat: command anvil-upgrade-init-addr * fix: prepare for upgrade, add custom init and getters * chore: update forge-std * deployment: add helper commands for anvil deployment; add script to deploy state updating contract * chore: gitignore generated file * chore: add custom solhit config in script folder * test: add cHr helper method to pretty print commission receivers * chore: remove redundant getter * chore: fix typos * chore: remove redundant scripts for deploying contracts * deployment: update prep-upgrade.js to take in an init contract address --------- Co-authored-by: Kevin Park <[email protected]> Co-authored-by: Theodore Georgas <[email protected]> Co-authored-by: Aleksandar Marinković <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Due to
web3jcompatibility issues, we need to reintroduceSimplePolicyInfostruct. There is an issue with smart contracts bindings for java, produced byweb3j. An error breaks the execution flow, when we want to fetch a struct which contains an array as a member variable. Ultimately, this prevents processing of theSimplePolicyPremiumPaidevent in backend and synchronisation of data between backend and blockchain.