CredComSoc / CreditCoopTech

Credit Coop Tech
Mozilla Public License 2.0
1 stars 1 forks source link

Add credit amounts where missing #125

Open tfwoodroofMCS opened 1 year ago

tfwoodroofMCS commented 1 year ago

In all instances where credit amounts appear either 'pollen' or 'pln' should be displayed alongside. See this comment for .json file to read from.

tfwoodroofMCS commented 1 year ago

Needs to be added in Cart next to amounts under 'Price' and 'Amount' (NB these should have been updated to 'Price/unit' and 'Price' respectively as per #26):

Image

tfwoodroofMCS commented 1 year ago

For 'Price' in preview screen of New item workflow:

Image

tfwoodroofMCS commented 1 year ago

For 'Price' and 'Amount' in My account -> My trades -> Pending Purchases (NB these should have been updated to 'Price/unit' and 'Price' respectively):

Image

tfwoodroofMCS commented 1 year ago

For 'Price' and 'Amount' in My account -> My trades -> Sales History (NB these should have been updated to 'Price/unit' and 'Price' respectively, and Sales History should now be Trading History):

Image

tfwoodroofMCS commented 1 year ago

After the 'total price' in the main body of the message generated when the user does not have sufficient credit to make a purchase request:

Image

tfwoodroofMCS commented 1 year ago

All fixed except for in Trading History.

Image

tfwoodroofMCS commented 1 year ago

Although have also just been logged in as test12 and it hadn't been fixed in their Pending Purchases (whereas it was fine for at least one other user, either test5 or test7, I'm not sure):

Image

yonasbek commented 1 year ago

Very weird!

tfwoodroofMCS commented 1 year ago

Have just tested with test15 and it's fine, so given that other recent changes don't seem to have been applied to test11 and test12, I think we can safely say that it's these users which are the problem.

However, credit units still need to be added to Trading History (see a few comments back), and I've also just noticed that on mobile the cart doesn't display units next to 'Total':

Image

tfwoodroofMCS commented 1 year ago

Spotted a further instance in a notification:

Image

credit unit needs to be added after blocked purchase request amount (900 in this instance).