Skip to content

Conversation

@cearachew
Copy link
Contributor

Adds the projections views for lesson hooks and exit tickets.

Exit tickets:
Screenshot 2026-02-05 at 11 29 43 AMScreenshot 2026-02-05 at 11 29 32 AM
Screenshot 2026-02-05 at 11 28 53 AM
Screenshot 2026-02-05 at 11 29 01 AM

Lesson hooks:
Screenshot 2026-02-05 at 12 25 18 PM

Links

  • Jira:

Testing story

Deployment strategy

Follow-up work

Privacy

Security

Caching

PR Creation Checklist:

  • Tests provide adequate coverage
  • Privacy impacts have been documented
  • Security impacts have been documented
  • Code is well-commented
  • New features are translatable or updates will not break translations
  • Relevant documentation has been added or updated
  • User impact is well-understood and desirable
  • Follow-up work items (including potential tech debt) are tracked and linked

@cearachew cearachew requested a review from etaderhold February 6, 2026 18:55
Copy link
Contributor

@etaderhold etaderhold left a comment

Choose a reason for hiding this comment

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

Looks great in general! Just a couple minor suggestions, plus it looks like Drone failed after you merged staging so make sure nothing's wrong there.

Comment on lines 526 to 527
// width: 960px;
// height: 918px;
Copy link
Contributor

Choose a reason for hiding this comment

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

please delete these if unused

width: 800px;
padding: 50px 40px 40px 40px;
flex-direction: column;
// justify-content: center;
Copy link
Contributor

Choose a reason for hiding this comment

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

here too

Comment on lines 630 to 639
color: var(--text-neutral-primary, #292F36);
text-align: center;
font-feature-settings: 'liga' off, 'clig' off;

/* Body/Body 1 - ExtraStrong */
font-family: var(--font-family-body, Figtree);
font-size: var(--font-size-body-lg, 20px);
font-style: normal;
font-weight: 700;
line-height: var(--font-line-height-body-lg, 28px); /* 140% */
Copy link
Contributor

Choose a reason for hiding this comment

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

indentation is wonky here

markdown={`**Question ${id + 1}:** ${question}`}
/>
</Typography>
{/* <BodyOneText>{`ANSWER: ${item.answer}`}</BodyOneText> */}
Copy link
Contributor

Choose a reason for hiding this comment

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

another commented-out bit that seems unneeded

displayExitTicket();
});

function displayExitTicket() {
Copy link
Contributor

Choose a reason for hiding this comment

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

Could we call this one displayLessonHook?

@cearachew cearachew marked this pull request as ready for review February 12, 2026 17:15
@cearachew cearachew merged commit 094b0cb into staging Feb 12, 2026
5 checks passed
@cearachew cearachew deleted the ceara/AITT-1174-artifact-projection-views branch February 12, 2026 18:55
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