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

Feature/pi 762 UI poc for cpm #480

Merged
merged 5 commits into from
Feb 13, 2025
Merged

Conversation

Rohoolio
Copy link
Contributor

No description provided.

@Rohoolio Rohoolio requested a review from a team as a code owner January 28, 2025 12:08
@Rohoolio Rohoolio force-pushed the feature/PI-762-ui_poc_for_cpm branch from 669307f to dceffd3 Compare January 28, 2025 12:08
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are these requirements for the 'real' UI or is this implying our test/demo UI also follows these standards?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Its just something that was generated as part of the process to help understand what was needed :) - its not reflective of the real thing

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tried to run this locally from the test_ui file but it didn't work :(

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah needed to cd into product_id_flow too, then run npm install before npm run dev


```bash
git clone <repository-url>
cd nhs-demo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this generic for future use?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IT WILL BE REMOVED FOREVER

@Rohoolio Rohoolio force-pushed the feature/PI-762-ui_poc_for_cpm branch from 1134f67 to a630b82 Compare February 10, 2025 16:27
@megan-bower4 megan-bower4 force-pushed the feature/PI-762-ui_poc_for_cpm branch from 4f73746 to f09b602 Compare February 12, 2025 16:31
@megan-bower4 megan-bower4 merged commit 431a93b into main Feb 13, 2025
21 of 22 checks passed
@megan-bower4 megan-bower4 deleted the feature/PI-762-ui_poc_for_cpm branch February 13, 2025 15:12
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.

2 participants