beckn / beckn-onix

beckn onix
MIT License
49 stars 27 forks source link

Add Reminder to Configure Layer 2 After BAP/BPP Setup in Beckn ONIX #210

Closed emmayank closed 2 months ago

emmayank commented 2 months ago

Description

By default, the BAP client does not have Layer 2 configuration schemas, which results in a 422 error when users try to make requests without configuring Layer 2. After successfully setting up BAP/BPP in Beckn ONIX, a message should be displayed to remind users to "Add Layer2 config before using the BAP/BPP" to prevent such errors.

Goals

Expected Outcome

  1. After successful setup of BAP/BPP, a message is displayed reminding users to add the Layer 2 config before using the BAP/BPP.
  2. The message should appear immediately after the success message to ensure visibility.

Acceptance Criteria

Mockups / Wireframes

N/A

Product Name

Beckn ONIX

Domain

N/A

Tech Skills Needed

Devops

Complexity

High

Category

Deployment

Sub Category

Installation

prasad-takale-eminds commented 2 months ago

I made the necessary changes to the code. Below is the screenshot of the BAP installation:

Image

After the changes, I connected with @vbabuEM and reviewed it. As per his suggestion, we need to prompt the user for layer configuration during the BAP/BPP installation. He has created a new issue for this: #222.

emmayank commented 2 months ago

Review Completed. this ticket is not needed. alternate ticket is created here #222