Validating UEI Specification and examples against the Beckn Core specification v 1.1.0

[rajaneeshk90]

Issue:

There are some components of the UEI specification which are not as per the Beckn core specification version 1.1.0
One such issue is highlighted below.

As per the UEI specification, "city" field should be enclosed inside context.location object.
Definition of the context in UEI specification can be found here: https://github.com/beckn/Unified-Energy-Interface/blob/main/api/dent.yaml#L2735
All the JSON examples in the UEI repository directly utilize the "city" field within the context.
More details of the issue can be found on the GitHub issue page of UEI repository: #17

As part of this task, we checked the UEI spec and example JSONs for any non-compliance with Beckn core spec version 1.1.0 and fixed the identified issues.

Change Made

• Removed the "city" field from the context object of all the example JSONs in the examples/ev-charging folder
• Added a "city" field in the context.location object of all the example JSONs in the examples/ev-charging folder
• removed order.items.fulfillments from on_init, on_confirm. on_status, on_update. Fulfilment info was already embedded in the order.fulfillments field.
• Adding add_ons info in the order.items in example JSONs.
• fixing order.items.quantity in the example JSONs.
• Changing support.order_id to support.ref_id in support API JSON
• Adding transaction_id in the payments.params

Related ticket: beckn/beckn-ui-workspace#485

[emmayank]

Hey @rajaneeshk90 , Please find my comments on the pull request below :

1. Search.intent has a location object which is not available in the core-v1.1 specification.
2. /track and /on_track API is not available, which is part of the core-v1.1 specification.
3. The examples that are defined in the YAML File (uei.yaml) are not as per the core-v1.1 specification.

I was hoping you could find the complete difference in the given comparison link (click here). Please make the required change and update the pull request.

Thanks
@emmayank

[rajaneeshk90]

Hi @emmayank

I have fixed the highlighted issues. Can you please review this PR.

There are more changes made to fix any non compliance in the UEI spec with the Beckn core specifications version 1.1.0.

Please let me know if any clarification is needed.

Thanks,
Rajaneesh