-
Notifications
You must be signed in to change notification settings - Fork 0
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
394: Add low-fi view ECT record #421
Conversation
f7fef0e
to
57dfe3c
Compare
row.with_key(text: "Induction status") | ||
|
||
# FIXME: This is a placeholder. The actual status should be displayed here. | ||
row.with_value(text: govuk_tag(text: "placeholder", colour: %w[grey green red purple orange yellow].sample)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should we use the status coming from the TRS API or are we going to have our own status values? @peteryates
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is where we'll show the overall induction status, which we will hold the 'truth' of in our database and push updates to TRS as and when they happen. We don't know what the possible values are yet though.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's keep the placeholder for now then
57dfe3c
to
fc696e9
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great!
Update ECT record view to match low-fi design:
Design:
Result: