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.

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

feat/fix(#1011): rewriting Barcode Configuration Admin page in React #1026

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

Conversation

adityadeshlahre
Copy link
Contributor

fix #1011

  • The PR title includes a brief description of the work done, including the Issue number if applicable.
  • The PR includes a video showing the changes for the work done.
  • The PR title follows conventional commit label standards.
  • The changes confirm to the OpenElis Global x3 Styleguide and design documentation.
  • The changes include tests or are validated by existing tests.
  • I have read and agree to the Contributing Guidelines of this project.

Summary

Rewriting admin page in ReactJS

Screenshots

FrontEnd

image

BackEnd

Getting perfect data from restController GET route working

image

Related Issue

[Add a link to the related issue or mention it here if applicable]

Other

[Add any additional information or notes here]

@adityadeshlahre
Copy link
Contributor Author

UIUPDATE

image
image

Now working on backend [ GET & POST ] request !
By Taking use of RESTCONTROLLER

@adityadeshlahre adityadeshlahre changed the title wip(#1011): rewriting Barcode Configuration Admin page in React feat/fix(#1011): rewriting Barcode Configuration Admin page in React May 2, 2024
@adityadeshlahre
Copy link
Contributor Author

Updated WorkFlow

barCode

At last commit added notification forget to shoot in video !

@adityadeshlahre adityadeshlahre marked this pull request as ready for review May 3, 2024 14:05
@github-actions github-actions bot added the merge conflict Merge Conflicts label May 8, 2024
Copy link

github-actions bot commented May 8, 2024

👋 Hi, @adityadeshlahre,
Conflicts have been detected against the base branch. Please rebase your branch against the base branch.


This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there.

@github-actions github-actions bot added merge conflict Merge Conflicts and removed merge conflict Merge Conflicts labels May 16, 2024
Copy link

👋 Hi, @adityadeshlahre,
Conflicts have been detected against the base branch. Please rebase your branch against the base branch.


This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there.

@mozzy11
Copy link
Collaborator

mozzy11 commented May 23, 2024

Thanks @adityadeshlahre any blocker or progress ??

@adityadeshlahre
Copy link
Contributor Author

adityadeshlahre commented May 23, 2024

Thanks @adityadeshlahre any blocker or progress ??

Hey @mozzy11
fixing merge conflicts and it will be ready to get merged

if any changes required let me know 🙂

@adityadeshlahre adityadeshlahre force-pushed the fix-1011-rewrite-barcode-configuration branch from 90af5b0 to b33ba7c Compare May 23, 2024 08:20
@github-actions github-actions bot removed the merge conflict Merge Conflicts label May 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Re Write BarCode Configuration Page in the new React UI
2 participants