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

Hcx communication #8178

Closed
wants to merge 29 commits into from
Closed

Hcx communication #8178

wants to merge 29 commits into from

Conversation

khavinshankar
Copy link
Member

Added HCX communication flow

Backend PR: ohcnetwork/care#1346

Further docs will be added later in docs repo

@khavinshankar khavinshankar requested a review from a team as a code owner July 19, 2024 06:26
Copy link

vercel bot commented Jul 19, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
care-storybook ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 23, 2024 0:02am

Copy link

netlify bot commented Jul 19, 2024

Deploy Preview for care-egov-staging ready!

Name Link
🔨 Latest commit b0ad0ba
🔍 Latest deploy log https://app.netlify.com/sites/care-egov-staging/deploys/66c87a0f08104d0008aed688
😎 Deploy Preview https://deploy-preview-8178--care-egov-staging.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

src/Components/HCX/ClaimCard.tsx Outdated Show resolved Hide resolved
src/Components/HCX/ClaimCardCommunication.tsx Outdated Show resolved Hide resolved
src/Components/HCX/SendCommunicationModal.tsx Outdated Show resolved Hide resolved
src/Redux/actions.tsx Outdated Show resolved Hide resolved
src/Redux/actions.tsx Outdated Show resolved Hide resolved
src/Redux/api.tsx Outdated Show resolved Hide resolved
Copy link

cypress bot commented Jul 19, 2024

CARE    Run #3181

Run Properties:  status check passed Passed #3181  •  git commit b0ad0ba671: Hcx communication
Project CARE
Branch Review hcx-communication
Run status status check passed Passed #3181
Run duration 02m 57s
Commit git commit b0ad0ba671: Hcx communication
Committer Khavin Shankar
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 126
View all changes introduced in this branch ↗︎

@nihal467
Copy link
Member

nihal467 commented Jul 23, 2024

@khavinshankar can you update the PR description for review purpose to understand what changed in this PR

@github-actions github-actions bot added the merge conflict pull requests with merge conflict label Jul 24, 2024
Copy link

👋 Hi, @khavinshankar,
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 removed the merge conflict pull requests with merge conflict label Jul 31, 2024
@github-actions github-actions bot added the merge conflict pull requests with merge conflict label Aug 8, 2024
Copy link

github-actions bot commented Aug 8, 2024

👋 Hi, @khavinshankar,
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 the stale label Aug 23, 2024
@github-actions github-actions bot removed the merge conflict pull requests with merge conflict label Aug 23, 2024
@khavinshankar
Copy link
Member Author

khavinshankar commented Aug 23, 2024

As the issues mentioned above involve working on the entire HCX module, it will be tackled in a separate PR and will be monitored in a new issue #8413. This way HCX Communication flow is not blocked for merging.

Ideally the above-mentioned issues will be tackled along with HCX plug seperation.

@github-actions github-actions bot added the merge conflict pull requests with merge conflict label Aug 23, 2024
Copy link

👋 Hi, @khavinshankar,
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 removed the merge conflict pull requests with merge conflict label Aug 23, 2024
@github-actions github-actions bot added the merge conflict pull requests with merge conflict label Sep 3, 2024
Copy link

github-actions bot commented Sep 3, 2024

👋 Hi, @khavinshankar,
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.

@nihal467
Copy link
Member

nihal467 commented Sep 9, 2024

@khavinshankar can you get this PR ready for review and testing

@bodhish
Copy link
Member

bodhish commented Sep 20, 2024

@khavinshankar is this required in the context of NHCX?

@khavinshankar
Copy link
Member Author

@bodhish Yes, this is one of the HCX flow but this PR can be closed as this is already part of #8420

@bodhish
Copy link
Member

bodhish commented Sep 20, 2024

Let's close it 🚀

@gigincg
Copy link
Member

gigincg commented Sep 21, 2024

Closing based on confirmation from @khavinshankar

@gigincg gigincg closed this Sep 21, 2024
@rithviknishad rithviknishad deleted the hcx-communication branch December 1, 2024 08:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changes required merge conflict pull requests with merge conflict reviewed reviewed by a core member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants