Closed moreal closed 9 months ago
This PR has 76
quantified lines of changes. In general, a change size of upto 200
lines is ideal for the best PR experience!
Was this comment helpful? :thumbsup: :ok_hand: :thumbsdown: (Email) Customize PullRequestQuantifier for this repository.
recipientAvatarAddr
은 받는 사람의 avatar address 이고, fungibleAssetValues[*].balanceAddr
은 Currency 의 종류에 따라 [agent address | avatar address] 입니다.
Bridge 를 통해 전달하고자 하는 경우 해당 주소를 vault 의 agent/avatar address 로 맞추어 보내고 있습니다. 완전히 상관 없는 주소가 들어오는 것에 대해 우려가 있다면 이렇게 맞추셔도 될 것으로 보입니다.
이 PR은:
fungibleAssetValues
를 unload하지 않으면fungibleItems
가 있더라도 아예 처리하지 않았습니다. 이제는fungibleItems
만 있더라도 처리하게 변경됩니다.fungibleAssetValues
를 받는balanceAddr
과는 무관하게 모두 memo를 통해 얻은agentAddress
로 주었습니다.fungibleAssetvalues
를 받는balanceAddr
에 따라서 bridge vault agentAddress로 주었다면 memo의 agentAddress로, vault avatarAddress로 주었다면 memo의 avatarAddress로 주게 변경됩니다. 그 외의fungibleAssetValues
는 무시합니다.recipientAvatarAddress
가 bridge vault avatarAddress과 다르다면 이벤트를 아예 무시했지만unload_from_my_garages
액션 스펙 상recipientAvatarAddress
와fungibleAssetValues[*].balanceAddr
은 전혀 무관한 값으로 이해해서 이제는recipientAvatarAddress
와 bridge vault avatarAddress가 다르더라도fungibleAssetValues
에 처리할 것이 있다면 처리합니다.