F 703 support multi recipient

[mohammadranjbarz]

related to Giveth/giveth-dapps-v2#703

[CarlosQ96]

Ignore this hahah

[mohammadranjbarz]

@MoeNick @CarlosQ96 @MohammadPCh

After merging this PR ( we are waiting for current release after that we would review and merge this)
The frontend should change two some web services:

createProject and updateProject

Instead of sending walletAddress you should send this in the body of request

 addresses: [
        {
          address: walletAddress,
          networkId: 1 (for mainnet) or 3( for ropsten) or 100 (for xDai),
        },
      ]

Getting projects

In every service that frontend gets project list or project detail , it would get address infoes as well with adding this to graphql query

 addresses {
          address
          isRecipient
          networkId
        }

And then based on each network that user selected on metamask frontend should use the address of that network, and make sure that the isRecipient is true for that address

[mohammadranjbarz]

@CarlosQ96 Please review it, but I keep it draft to not merge it until the frontend guys @MohammadPCh or @MoeNick tell us to merge it, because this PR changes are not backward compatible and staging may break if we merge it without implementing frontend side

[CarlosQ96]

#525 (comment) I think you mentioned the projectByAddress mutation is no longer used. We should delete it as it would be incomplete with this Pr, it would need to search by related addresses too.

Same for project query. As we do not user repository to fetch project.

[CarlosQ96]

Sql injection can be done here, escape the address on the raw query.

[CarlosQ96]

Quick question, is the previous purple address list lost? gotta reintroduce them in the adminbro?

[mohammadranjbarz]

Quick question, is the previous purple address list lost? gotta reintroduce them in the adminbro?

Yeah we would lose them, but now there isn't any purple addresses in our DB and I told Ashley to not add any addresses until we merge this, so it would not make a problem

[CarlosQ96]

I think we can skip deleting the other mutations for now. The sql injection is fixed.

[CarlosQ96]

Awesome job man! All issues fixed. I think we only have to coordinate with @mateodaza to merge. He is wanting to integrate.