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

[SYNSD-893] Add query for disjointed name #31

Merged
merged 1 commit into from
May 14, 2024

Conversation

BryanFauble
Copy link
Contributor

Problem:

  • Synapse allows an entity name to be disjointed from the name of the file. This can lead to some problems when the downloadAs name (The name of the file) overlaps with other files in the folder - and when the name of the FileEntity appears to be correct.

Solution:

  • Adding in a small query I used to identify which Files in a specific folder had a DownloadAs name which did not match the name of the entity. We had used this to pass along information over to the collaborator to resolve the issue.

I also added a small script via this commit to the examples section that contains how to update all files in a folder to match the downloadAs to the name of the entity: Sage-Bionetworks/synapsePythonClient@ed00ef5

Copy link

sonarcloud bot commented Mar 28, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@BryanFauble
Copy link
Contributor Author

@thomasyu888 Is this the best location to put these 1-off scripts. Is there something else you had in mind, or perhaps if we should create another folder to contain these?

@thomasyu888 thomasyu888 merged commit 3efd537 into dev May 14, 2024
3 checks passed
@thomasyu888 thomasyu888 deleted the synsd-893-disjointed-file-name-to-entity-name branch May 14, 2024 07:10
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