Skip to content

Fix TypeScript compiler issues#280

Merged
paulrobertlloyd merged 4 commits into
mainfrom
use-nhsuk-prototype-kit-package-typescript
May 12, 2026
Merged

Fix TypeScript compiler issues#280
paulrobertlloyd merged 4 commits into
mainfrom
use-nhsuk-prototype-kit-package-typescript

Conversation

@colinrotherham
Copy link
Copy Markdown
Collaborator

Slowly getting rid of red squiggles

@paulrobertlloyd paulrobertlloyd temporarily deployed to mavis-protot-use-nhsuk--b6gkek May 8, 2026 16:27 Inactive
@paulrobertlloyd
Copy link
Copy Markdown
Collaborator

@colinrotherham You might want to rebase this with main; only 4 red squiggles left (without strict: true)

@colinrotherham colinrotherham force-pushed the use-nhsuk-prototype-kit-package-typescript branch from c76d103 to 07b97f1 Compare May 12, 2026 07:43
@colinrotherham colinrotherham temporarily deployed to mavis-protot-use-nhsuk--b6gkek May 12, 2026 07:43 Inactive
@colinrotherham
Copy link
Copy Markdown
Collaborator Author

Just getting this one now @paulrobertlloyd

Doesn't appear to be .code property on the Team class

app/controllers/session.js:589:33 - error TS2339: Property 'code' does not exist on type 'Team'.

589       team = Team.findOne(team?.code || '001', data)
                                    ~~~~

@colinrotherham colinrotherham changed the title [WIP] Review TypeScript compiler issues Fix TypeScript compiler issues May 12, 2026
@colinrotherham colinrotherham marked this pull request as ready for review May 12, 2026 07:45
@paulrobertlloyd paulrobertlloyd force-pushed the use-nhsuk-prototype-kit-package-typescript branch from 07b97f1 to 92f97b7 Compare May 12, 2026 08:50
@paulrobertlloyd paulrobertlloyd temporarily deployed to mavis-protot-use-nhsuk--b6gkek May 12, 2026 08:50 Inactive
@paulrobertlloyd paulrobertlloyd force-pushed the use-nhsuk-prototype-kit-package-typescript branch from 92f97b7 to 4faba42 Compare May 12, 2026 08:51
@paulrobertlloyd paulrobertlloyd temporarily deployed to mavis-protot-use-nhsuk--b6gkek May 12, 2026 08:51 Inactive
@paulrobertlloyd
Copy link
Copy Markdown
Collaborator

Doesn't appear to be .code property on the Team class

Ooops, that should now be .id. Fixed!

@paulrobertlloyd paulrobertlloyd temporarily deployed to mavis-protot-use-nhsuk--b6gkek May 12, 2026 08:59 Inactive
@paulrobertlloyd
Copy link
Copy Markdown
Collaborator

Thanks @colinrotherham ❤️

@paulrobertlloyd paulrobertlloyd merged commit f95a2b2 into main May 12, 2026
1 check passed
@paulrobertlloyd paulrobertlloyd deleted the use-nhsuk-prototype-kit-package-typescript branch May 12, 2026 09:00
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