Skip to content

Added length cap to sketch name #944

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

Merged
merged 1 commit into from
Mar 20, 2019
Merged

Conversation

siddhant1
Copy link
Contributor

I have verified that this pull request:

  • [x ] has no linting errors (npm run lint)
  • [ x] is from a uniquely-named feature branch and has been rebased on top of the latest master. (If I was asked to make more changes, I have made sure to rebase onto master then too)
  • [x ] is descriptively named and links to an issue number, i.e. Fixes #123

ref #568 (comment)

@catarak
Copy link
Member

catarak commented Mar 20, 2019

i think this is a great start and i can merge this in as it is now. however, i wonder if this should also be included server-side? someone could just delete the maximum length in the html and save the project.

@catarak
Copy link
Member

catarak commented Mar 20, 2019

i'm going to merge and leave #568 open.

@catarak catarak merged commit 98d7933 into processing:master Mar 20, 2019
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