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

Add SynchWeb user permission for proposal directories #44

Merged
merged 5 commits into from
Feb 12, 2024

Conversation

stuartcampbell
Copy link
Collaborator

@stuartcampbell stuartcampbell commented Feb 12, 2024

Added the missing SynchWeb user read permissions if the beamline uses the SynchWeb service.

The method of determining if a beamline uses SynchWeb is to look at the services field in the beamline document which is a list of services that the beamline uses. If one exists with the name of synchweb then we assume that the beamline is using SynchWeb

This resolves #40

Copy link
Collaborator

@nmaytan nmaytan left a comment

Choose a reason for hiding this comment

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

LGTM

@stuartcampbell stuartcampbell merged commit a1642c4 into NSLS2:main Feb 12, 2024
1 check passed
@stuartcampbell stuartcampbell deleted the synchweb-permissions branch February 12, 2024 18: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.

AMX/FMX/NYX lacking synchweb user right
2 participants