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: Periods not allowed in task name #98

Open
rclough opened this issue Jun 12, 2014 · 3 comments
Open

Bug: Periods not allowed in task name #98

rclough opened this issue Jun 12, 2014 · 3 comments
Labels

Comments

@rclough
Copy link
Collaborator

rclough commented Jun 12, 2014

Periods apparently aren't allowed in the task name. This should either be checked by JS on the front end, or at least we should have more intelligent error reporting around it.

Error from stderr:
The dotted field 'Name with.period' in 'dependencies.Name with.period' is not valid for storage.

@rclough
Copy link
Collaborator Author

rclough commented Jun 12, 2014

after a hard refresh it appears the job gets added but it causes errors on all further commands, haven't tested for other issues.

@thieman
Copy link
Owner

thieman commented Jun 13, 2014

I'm guessing this only affects the Mongo backend, but we'll need to protect against periods regardless to keep things consistent.

@thieman thieman added the bug label Jun 13, 2014
@rclough
Copy link
Collaborator Author

rclough commented Jun 13, 2014

Confirming this is just with mongo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants