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

chore(tests): Update test snapshots 24-10-2024 #5354

Merged
merged 1 commit into from
Oct 24, 2024

Conversation

dfe-sdt
Copy link
Collaborator

@dfe-sdt dfe-sdt commented Oct 24, 2024

This PR was auto-generated on 24-10-2024 by create-pull-request due to visual differences identified

Copy link
Collaborator

@cjrace cjrace left a comment

Choose a reason for hiding this comment

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

Seems like it's all still there so happy to approve, though curious if there's any reason why the ordering might have changed? @N-moh any ideas?

@N-moh
Copy link
Collaborator

N-moh commented Oct 24, 2024

Seems like it's all still there so happy to approve, though curious if there's any reason why the ordering might have changed? @N-moh any ideas?

After 'topics' deletion, the publications got reordered alphabetically. Previously the order would have been different as they were sub-grouped under topics . This change was deployed this morning, so we are seeing snapshots differences .

@N-moh N-moh merged commit 70700ed into dev Oct 24, 2024
7 checks passed
@N-moh N-moh deleted the chore/update-test-snapshots branch October 24, 2024 12:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants