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

issue: external storage naming #11755 #12049

Closed

Conversation

MasterOfDeath
Copy link
Contributor

@MasterOfDeath MasterOfDeath commented Oct 25, 2018

Validation for existing mountpoint path
fixes #11755

@@ -43,6 +43,7 @@
"Storage with ID \"%d\" not found" : "Storage with ID \"%d\" not found",
"Invalid backend or authentication mechanism class" : "Invalid backend or authentication mechanism class",
"Invalid mount point" : "Invalid mount point",
"Invalid path of mountpoint" : "Invalid path of mountpoint",
Copy link
Member

Choose a reason for hiding this comment

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

This is not required. Our translation scrips sync this auytmaitcally to transifex for translations

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Should I remove it from PR?

Copy link
Member

Choose a reason for hiding this comment

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

Yes - the automatic script will update it and the translations are stored on transifex anyways.

Signed-off-by: Rinat Gumirov <rinat.gumirov@mail.ru>
@MasterOfDeath MasterOfDeath force-pushed the external-storage-naming branch from 4bb6407 to 1d94492 Compare October 29, 2018 19:42
@@ -45,6 +45,7 @@ OC.L10N.register(
"Storage with ID \"%d\" not found" : "Storage with ID \"%d\" not found",
"Invalid backend or authentication mechanism class" : "Invalid backend or authentication mechanism class",
"Invalid mount point" : "Invalid mount point",
"Invalid path of mountpoint" : "Invalid path of mountpoint",
Copy link
Member

Choose a reason for hiding this comment

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

Same for this.

@MorrisJobke
Copy link
Member

@icewind1991 I would like to have your feedback on this one.

@blizzz blizzz added this to the Nextcloud 15 milestone Nov 1, 2018
@MorrisJobke MorrisJobke mentioned this pull request Nov 6, 2018
29 tasks
@MorrisJobke MorrisJobke modified the milestones: Nextcloud 15, Nextcloud 16 Nov 7, 2018
@rullzer
Copy link
Member

rullzer commented Feb 4, 2019

@icewind1991 ping

@MorrisJobke MorrisJobke modified the milestones: Nextcloud 16, Nextcloud 17 Mar 6, 2019
@MorrisJobke MorrisJobke mentioned this pull request Jul 17, 2019
28 tasks
@rullzer
Copy link
Member

rullzer commented Sep 15, 2019

I'm going to clos this here.
@icewind1991 if you still want this please reopen.

@rullzer rullzer closed this Sep 15, 2019
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.

external storage naming
4 participants