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

[Bug] Workspace name can't contain uppercase #361

Open
noraj opened this issue Jun 7, 2019 · 4 comments
Open

[Bug] Workspace name can't contain uppercase #361

noraj opened this issue Jun 7, 2019 · 4 comments

Comments

@noraj
Copy link

noraj commented Jun 7, 2019

Description

An alert tells me Error: Workspace name can not contain white spaces or start with uppercase. when I try to name the workspace testWorkspace but testWorkspace doesn't start with an uppercase and contains no white-space.

This may due to a bad regex matching white-space at any position instead of checking the first char only.

Issue Type

  • Bug Report

Faraday version

Faraday v3.7.0

Component Name

I don't know.

Steps to reproduce

  • Create a new workspace
  • Name it testWorkspace
  • You can't save it

Expected results

I could name a workspace testWorkspace.

Debugging tracebacks (current results)

It is more a bad implementation than a bug so there are not debug logs.

Screenshots

image

Environment information

Configuration files

No needed. I changed nothing pure default.

OS

Provide information on your operating system. Example:

$ cat /etc/lsb-release
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=16.10
DISTRIB_CODENAME=yakkety
DISTRIB_DESCRIPTION="Ubuntu 16.10"

🤣 /etc/lsb-release? The standard file is /etc/os-release. #362

cat /etc/os-release 
NAME="Arch Linux"
PRETTY_NAME="Arch Linux"
ID=arch
BUILD_ID=rolling
ANSI_COLOR="0;36"
HOME_URL="https://www.archlinux.org/"
DOCUMENTATION_URL="https://wiki.archlinux.org/"
SUPPORT_URL="https://bbs.archlinux.org/"
BUG_REPORT_URL="https://bugs.archlinux.org/"
LOGO=archlinux
@noraj
Copy link
Author

noraj commented Jun 7, 2019

PS: Should I also fill a feature request to have Workspace name supports spaces and uppercase letter at first char?

@llazzaro
Copy link
Contributor

llazzaro commented Jun 7, 2019

@noraj It's ok to leave it here. We may add support for upper on first char. For white spaces we need to add a slug, since workspace name is used on the api urls.

@noraj
Copy link
Author

noraj commented Jun 7, 2019

@llazzaro Or you can associate a UID to each workspace and use the UID in the API. The UID is unique and never change, then you can associate whatever name at it and even rename the workspace.

@llazzaro
Copy link
Contributor

We have been discussing this issue with the dev team. The problem of using UID is backwards compatbility.
We could start to use the UID for new workspaces and use the current workspace name for old workpsaces. A lot of peopel are usign integrations with the faraday API.

The restriction on the workspace name cames from the couchdb era, where the workspace name was the couchdb database name.

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

No branches or pull requests

2 participants