Skip to content

Conversation

@jimmyfagan
Copy link
Contributor

@jimmyfagan jimmyfagan commented Sep 24, 2024

JIRA Ticket:
BB2-3391

What Does This PR Do?

This PR adjusts the rendering of the FE sample client for C4DIC. There is still more work to be done, but this at least sets up the general format of a card that resembles the designs in Figma.

What Should Reviewers Watch For?

Especially since I don't have a lot of FE experience, if any of this significantly breaks norms or is bad practice, please point that out. Anything that definitely needs to be done in a future PR that is not already marked with a TODO should be called out as well.

Validation

I've verified this by running locally against my local server which is running the feature branch of the web server repo. To do this, make sure your HOST_IP in docker-compose.yml matches your local IP (can be viewed in your wifi settings under the details of your current connection).

Results should look like this:
image

@jimmyfagan jimmyfagan marked this pull request as ready for review September 24, 2024 18:43
Copy link
Contributor

@JFU-NAVA-PBC JFU-NAVA-PBC left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link

@noor1027 noor1027 left a comment

Choose a reason for hiding this comment

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

LGTM

@jimmyfagan jimmyfagan merged commit a09c510 into feature/c4dic Sep 24, 2024
@jimmyfagan jimmyfagan deleted the feature/c4dic-jimmy-bb2-3391 branch September 24, 2024 21:45
@jimmyfagan jimmyfagan restored the feature/c4dic-jimmy-bb2-3391 branch October 8, 2024 22:21
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.

4 participants