Remove automatic supply queue push

[MathisGD]

https://morpholabs.slack.com/archives/C02N7CSP91C/p1702569747842329

and fixes #352

[Rubilmax]

this discussion needs to be resolved first: https://github.com/morpho-org/metamorpho/pull/351/files#r1427738018

[adhusson]

This also fixes MM-3 from my report (but duplicates can still be set manually, and so maxDeposit can still be wrong).

[QGarchery]

Suggested change

	if (newCap > 0) {
	if (!isEnabled) {
	if (newCap > 0 && !isEnabled) {

[StErMi]

The PR seems OK. Consider documenting the de-sync between the withdrawQueue and supplyQueue (before, when a market was added, both the supply and withdraw queue were configured and pushed).

You could have the whole withdrawQueue fully configured, but no market from which you can supply to.

[QGarchery]

You could have the whole withdrawQueue fully configured, but no market from which you can supply to.

Yes this is a wanted behavior: some vault curators will prefer reallocating every time instead of using the supply queue.

Consider documenting the de-sync between the withdrawQueue and supplyQueue (before, when a market was added, both the supply and withdraw queue were configured and pushed).

I think it's ok considering that there is already other functions that could desync the two queues (which seems natural since the code has 2 queues, so it would make sense if they are not always the same)