Closed (fixed)
Project:
Commerce Point of Sale (POS)
Version:
8.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Reporter:
Created:
21 Aug 2017 at 15:27 UTC
Updated:
20 Sep 2017 at 00:15 UTC
Jump to comment: Most recent, Most recent file

Comments
Comment #2
rakesh.gectcrComment #3
rakesh.gectcr[Wed Aug 23 20:20:13.317053 2017] [php7:notice] [pid 12357] [client 127.0.0.1:51666] Uncaught PHP Exception Drupal\\Core\\Entity\\EntityMalformedException: "The entity does not have an ID." at /Users/rakeshjames/Sites/mystore/web/core/lib/Drupal/Core/Config/Entity/ConfigEntityStorage.php line 248, referer: http://mystore.lo/admin/commerce/config/currency_denominations/addComment #4
rakesh.gectcrAre we expecting
denominationsfield with Add more Ajax button ?Comment #5
smccabe commentedThat would probably make the most sense, although it could also just be a separate add form. The current d7 version just uses a very standard add/edit/delete setup, but it also isn't a config entity.
In short, whichever you feel is easier/better is fine with me.
Comment #6
rakesh.gectcrComment #7
rakesh.gectcrComment #8
rakesh.gectcrComment #9
rakesh.gectcrneed to add the changes from CurrencyDenominations file as well
Comment #10
rakesh.gectcrComment #11
swickham commentedHere's what I could find that I think need work:
Comment #12
rakesh.gectcr@swickham Thank you, for the review.
I have done all the points which you mentioned above except
We will work on this after the alpha release.
I have created a separated issue as a child of this So that we will not miss it.
#2906918: The "Remove one" button makes it difficult to manage denominations in the form
Comment #13
swickham commentedLooks good.
Comment #15
smccabe commented