Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Edit allocated devices: API Integration #1778

Open
wants to merge 23 commits into
base: develop
Choose a base branch
from
Open

Edit allocated devices: API Integration #1778

wants to merge 23 commits into from

Conversation

Shruti-Apte
Copy link
Contributor

Notion

What

  • Enabled edit devices page
  • API integration for edit devices
  • Now users can edit or delete previous devices and add new devices with a single request.
  • Added validations and error messages for each device.
  • Refactoring: Created a DeviceForm common component to use it on both mobile and laptop screens.

Note:

“Apoorv and others added 21 commits April 22, 2024 15:15
@apoorv1316
add insurance details
ae0f4dd
@apoorv1316
api for bulk operations
01e3c32
@apoorv1316
add insurance details while adding device
df608f5
@apoorv1316
fix-rubocop
04a2169
@apoorv1316
Merge branch 'develop' into devices-api-bulk
d7f8f7b
Merge branch 'devices-api-bulk' of https://github.com/saeloun/miru-web …
b69ea9f 
…into allocated_devices_edit_api

pulled new backend changes
@apoorv1316
add condition for insurance validation
4f7aebf
Merge branch 'devices-api-bulk' of https://github.com/saeloun/miru-web …
91cbddf 
…into allocated_devices_edit_api

pulled new changes
@apoorv1316
fix spec and add service specs
5a1d557
Insurance details UI and API integration
127cdbd
Merge branch 'devices-api-bulk' of https://github.com/saeloun/miru-web …
cc6c63e 
…into allocated_devices_edit_api

pulled the latest backend changes
Merge branch 'develop' into allocated_devices_edit_api
7031e44 
merge with develop
@apoorv1316
refactor
9d5bda3
@apoorv1316
Revert "refactor"
9f4784d 
This reverts commit 9d5bda3.
@apoorv1316
refactor code
8586773
Merge branch 'devices-api-bulk' of https://github.com/saeloun/miru-web …
8264bc9 
…into allocated_devices_edit_api

pulled new backend changes
date bug fixed
adae399
merge with develop and conflicts resolved
5ce2192
insurance_bought_date updated to insurance_activation_date
82cbb47
Merge branch 'develop' into allocated_devices_edit_api
b7de777 
merge with develop
bug fixes
175c215
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nisusam nisusam Awaiting requested review from nisusam

@apoorv1316 apoorv1316 Awaiting requested review from apoorv1316

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Shruti-Apte @apoorv1316