Skip to content

Update moveFundsBetweenPots usage #3820

@kronosapiens

Description

@kronosapiens

During testing of JoinColony/colonyNetwork#1193 (see comment) it appears as though the front-end is using a deprecated version of the moveFundsBetweenPots function. Specifically, it is using this version:

moveFundsBetweenPots(uint256,uint256,uint256,uint256,uint256,uint256,address)

Which has been deprecated for security purposes. The new version is

moveFundsBetweenPots(uint256,uint256,uint256,uint256,uint256,uint256,uint256,uint256,address)

Featuring two additional uint256 arguments at the front-end, linking the permission domain to the "parent" domain of the two funding pots being affected.

  function moveFundsBetweenPots(
    uint256 _permissionDomainId,
    uint256 _childSkillIndex,
    uint256 _domainId,
    uint256 _fromChildSkillIndex,
    uint256 _toChildSkillIndex,
    uint256 _fromPot,
    uint256 _toPot,
    uint256 _amount,
    address _token
  )

Usage should be updated prior to merging of JoinColony/colonyNetwork#1193.

Metadata

Metadata

Assignees

Labels

bugSomething isn't working

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions