-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature/1 ordering #6
Open
dladlk
wants to merge
71
commits into
main
Choose a base branch
from
feature/1-ordering
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…e, show progress on adding
…Data, add/remove/change quantity per product. Tried to use a data class for this - but had issues with React.useState on object class...
… in React.useState
…atus to BasketData; pass changeBasket and basketData through intermediate components to AddToBasket via ...props without explicitly naming parameters.
… buttons, add mouse-over titles to buttons
… `ForwardRef(GridList)` - change from string "360" to {360}
…y" prop." Add several keys to render objects
…ing dependency: 'loadProducts'. Either include it or remove the dependency array react-hooks/exhaustive-deps" - but give up.
…dding OrderHeader component to BasketPage
…ding, hide refresh on sending, show progress bar, temporary add delay seeing how it looks on long sending
…state to keep it, but update global class object inside input onchange, use local state per each input, avoid parameter passing to each DataInput by using React.cloneElement with expanded list of parameters inside DataBlock; build label by field name - split by uppercase and convert first letter to upper. See comments in commit for reasons.
…neration and integrations configuration
…venience, change top icon of basket to just ORDER text, remove artificial delay in basket adding
…highlight failed products
…upplied to `ForwardRef(Fab)`, expected one of ["default","inherit","primary","secondary"].
…PHAX UBL2.1 library into Mongo - done with intermediate solution where everything is converted to UTC; implement loading of basket details together with all order JSON model; add refresh on basket history
…to a separate file
…sket, with links to products on order details (now points to orderLine.item.id - wrong)
…asy to navigate to a product in order details
….jackson.default-property-inclusion=NON_EMPTY - e.g. for sent Order details
… the problem on Linux
… com.helger.ubl:ph-ubl21 The issue was that IssueDate/IssueTime were serialized as below: "issueDate": { "value": { "dayOfYear": 120, "dayOfWeek": "SATURDAY", "year": 2022, "month": "APRIL", "monthValue": 4, "dayOfMonth": 30, "asString": "2022-04-30Z", "offset": "Z" }, "valueLocal": "2022-04-30" }, "issueTime": { "value": { "nano": 639000000, "hour": 21, "minute": 39, "second": 42, "asString": "21:39:42.639Z", "offset": "Z" }, "valueLocal": "21:39:42.639" } Adding custom serializers to MongoConverterConfig looked like did not help - but it was because serialization in Spring Controller response used Jackson serialization! Instead of writing own binding of Order (and potentially future other UBL 2.1 document types), instead of object response with Jackson serialization, MongoTemplate direct query is added to OrderService to load raw Mongo JSON in BSON format, which gives a benefit - same presentation in Mongo and on GUI.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.