Skip to content

Conversation

@AlexJacksonDS
Copy link
Contributor

Round 2 with actually correct filter text. Should have checked more closely that they were actually working.

Copy link
Contributor

@DanBloxham-sw DanBloxham-sw left a comment

Choose a reason for hiding this comment

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

This is looking good, but I think the diagnostic one will need to be changed from "HasDiagnostic,DiagnosticsScore" to "HasDiagnostic,DiagnosticScore" (making "Diagnostics" singular before the score)

@AlexJacksonDS
Copy link
Contributor Author

Yep, good spot. Missed the extra s

@stellake
Copy link
Contributor

Just double checking if we've made sure all refactored properties work as expected?

@DanBloxham-sw
Copy link
Contributor

When I tested previously, each one in that list worked except for the three being changed.

@AlexJacksonDS
Copy link
Contributor Author

And I've checked that these ones I've just changed work locally

Copy link
Contributor

@stellake stellake left a comment

Choose a reason for hiding this comment

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

Given that all the refactored search properties are tested, happy for merge! 👍

@AlexJacksonDS AlexJacksonDS merged commit 97b37a4 into master Jun 30, 2021
@stellake stellake deleted the HEEDLS-415-admin-page-pagination branch July 1, 2021 10:54
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.

3 participants