Skip to content

Conversation

@stuartcampbell
Copy link
Collaborator

@stuartcampbell stuartcampbell commented Jul 14, 2025

This simply changes the nsls2_redhat_satellite_location_name field in the Beamlines collection from a str to a list[str].

I have also included a simple script in scripts directory to update the documents in the database - this will need to be done

  1. on the nsls2core-development database when merged into main
  2. on the nsls2core-production database before a new release is pushed to production.

This resolves #180

EDIT: Actually as I don't think anything is actively using this field - all databases can be updated once this is merged.

@stuartcampbell stuartcampbell self-assigned this Jul 14, 2025
@stuwilkins stuwilkins merged commit a75d42e into NSLS2:main Jul 14, 2025
4 checks passed
@stuartcampbell stuartcampbell deleted the beamline-satellite-location-change branch July 14, 2025 20:42
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.

Satellite location should be list

2 participants