fix(request-client): make creation refresh synchronous

[alexandre-abrioux]

Issue

During creation of a request we do an await request.refresh() right before returning the request object. This creates a call to the request node that queries the pending request's data.

However the request is an EventEmitter, so error or confirmed events can be emitted while the pending request's data is being fetched. This causes consuming code to miss those events.

Description of the changes

Two solutions:

1. Either we remove the await request.refresh() all together. It is not a big deal knowing that another refresh is done after the confirmation happens anyway here:
   

   requestNetwork/packages/request-client.js/src/api/request.ts

   Lines 106 to 113 in 7efaec7

   	originalEmitter
   	.on('confirmed', async () => {
   	try {
   	this.emitter.emit('confirmed', await this.refresh());
   	} catch (error) {
   	originalEmitter.emit('error', error);
   	}
   	})

   
   However it means we would remove having a "pending" state in the request.

2. Or we could refresh the request by injecting the data that we already have = the data that was sent to the request node = without querying the request node = in a synchronous way.

I went for the second option here.