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

feat: make large file access warning text configurable with largeData… #1517

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

Junjiequan
Copy link
Contributor

…FileAccessInstruction.

Description

Change for the SciCatProject/frontend#1666

Motivation

Fixes

  • Bug fixed (#X)

Changes:

  • changes made

Tests included

  • Included for each change/fix?
  • Passing?

Documentation

  • swagger documentation updated (required for API changes)
  • official documentation updated

official documentation info

@Junjiequan Junjiequan marked this pull request as ready for review November 20, 2024 14:18
@@ -120,6 +120,7 @@
"searchPublicDataEnabled": true,
"searchSamples": true,
"sftpHost": "login.esss.dk",
"largeDataFileAccessInstruction": "",
Copy link
Contributor

Choose a reason for hiding this comment

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

maxFileSizeWarning ?

@Junjiequan Junjiequan force-pushed the SWAP-4247-change-sftp-text-on-scicat branch from 7997568 to 80b00b4 Compare November 27, 2024 15:01
@Junjiequan Junjiequan force-pushed the SWAP-4247-change-sftp-text-on-scicat branch from 80b00b4 to b959770 Compare November 28, 2024 18:20
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