Skip to content
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

Added a feature where the user table only renders when a string is present in the search box #3511

Conversation

domducsutcliffe
Copy link
Contributor

@domducsutcliffe domducsutcliffe commented Jan 26, 2023

Issue: https://crowncommercialservice.atlassian.net/jira/software/c/projects/FMFR/boards/71?modal=detail&selectedIssue=FMFR-1338

I have included an additional line in the background of a feature test, to reflect the new behaviour.

@domducsutcliffe domducsutcliffe force-pushed the FMFR-1338-enhancements-to-user-search-feature-user-admin-section branch from eda4fde to 188c61f Compare January 26, 2023 16:22
Copy link
Collaborator

@tim-s-ccs tim-s-ccs left a comment

Choose a reason for hiding this comment

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

Just a couple of things for you to think about plus one change (add the step to more feature tests)

@domducsutcliffe domducsutcliffe force-pushed the FMFR-1338-enhancements-to-user-search-feature-user-admin-section branch from e18097c to c1c2f68 Compare January 27, 2023 13:24
@domducsutcliffe domducsutcliffe force-pushed the FMFR-1338-enhancements-to-user-search-feature-user-admin-section branch from 0bdc1f8 to 6226877 Compare January 27, 2023 13:36
…ure test to be more precise by targeting the css in the rendered partial
@domducsutcliffe
Copy link
Contributor Author

Believe I've made all the needed changes @tim-s-ccs

@domducsutcliffe domducsutcliffe merged commit 7972062 into master Jan 30, 2023
@domducsutcliffe domducsutcliffe deleted the FMFR-1338-enhancements-to-user-search-feature-user-admin-section branch January 30, 2023 10:50
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