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.

Sign up for GitHub

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

Jump to bottom

Prevent reserved words from being used as field names #3

Closed
JedWatson opened this issue Aug 7, 2013 · 2 comments
Closed

Prevent reserved words from being used as field names #3

JedWatson opened this issue Aug 7, 2013 · 2 comments
Assignees
@JedWatson
Labels
enhancement

Comments

@JedWatson
Copy link
Member

id, _id, _, etc.
@ghost ghost assigned JedWatson Aug 7, 2013
estilles pushed a commit to estilles/keystone that referenced this issue Jul 29, 2014
attempting to fix test failure on Travis (keystonejs#3)
5aa1144
estilles pushed a commit to estilles/keystone that referenced this issue Aug 4, 2014
Merge branch 'feature/adding-optional-list-audit-meta' into develop
f29915c 
* feature/adding-optional-list-audit-meta:
  attempting to fix test failure on Travis (keystonejs#3)
  attempting to fix test failure on Travis (#2)
  attempting to fix test failure on Travis
  added unit test for 'audit meta' feature
  refactored 'audit meta' feature into a schemaPlugin (to improve testability)
  added optional audit meta fields to track createdBy/updatedBy user

Conflicts:
	lib/list.js
	lib/updateHandler.js
@JedWatson
Copy link
Member Author

Still a good idea @sebmck

@sebmck sebmck closed this as completed in d7de98b Jan 19, 2015
@webteckie
Copy link
Contributor

I ran into issues with: "action", "domain", and "collection" as well. Some of that may be due to mongoose...not sure.

JedWatson pushed a commit that referenced this issue Mar 18, 2015
@montmanu
Merge pull request #3 from keystonejs/master
3f09be4 
Merge back keystonejs/keystone
@ericelliott ericelliott mentioned this issue Jan 6, 2016
Closed
@raviinder raviinder mentioned this issue Dec 8, 2016
Closed
Noviny pushed a commit that referenced this issue Mar 14, 2017
@KyleAMathews
Merge pull request #3 from Noviny/add-website
e4ed8cd 
Add setting up draft
dominikwilkowski pushed a commit that referenced this issue Aug 2, 2018
Merge pull request #3 from nathansimpson95/fourOhFour
ad91894 
Four oh four
UnGast pushed a commit to UnGast/keystone that referenced this issue Dec 22, 2018
@Tathanen
Merge pull request keystonejs#3 from keystonejs/master
0e4dd88 
Pulling from keystone/master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@JedWatson JedWatson

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@JedWatson @webteckie