Skip to content

Commit

Permalink
Remove knex adapter (#5278)
Browse files Browse the repository at this point in the history
  • Loading branch information
timleslie authored Mar 30, 2021
1 parent 8665cfe commit 5777fc6
Show file tree
Hide file tree
Showing 19 changed files with 17 additions and 2,480 deletions.
2 changes: 1 addition & 1 deletion packages-next/admin-ui/TODO.md
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
- [x] mongoId in fields package
- [x] `idField` option
- [x] disallow `id` field in fields object
- [x] default `idField` to `autoIncrement` for Knex & Prisma and `mongoId` for Mongoose with fieldMode: 'hidden' for itemView and createView
- [x] default `idField` to `autoIncrement` for Prisma with fieldMode: 'hidden' for itemView and createView
- [x] Remove usage of label field in Admin UI and remove it from the GraphQL schema(~~with a feature flag to turn it back on~~ and allow Keystone fields named `_label_` so users can create a virtual field for `_label_`)
- [ ] Ensure support for field descriptions in the create and item views
- [ ] Figure out what to support for custom list views
Expand Down
1 change: 0 additions & 1 deletion packages-next/keystone/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,6 @@
"@types/express": "^4.17.11",
"@types/fs-extra": "^9.0.9",
"@types/graphql-upload": "^8.0.4",
"@types/keystonejs__adapter-knex": "^6.3.2",
"@types/keystonejs__adapter-mongoose": "^5.1.2",
"@types/keystonejs__keystone": "^7.0.1",
"@types/pluralize": "^0.0.29",
Expand Down
2 changes: 0 additions & 2 deletions packages/adapter-knex/.npmignore

This file was deleted.

776 changes: 0 additions & 776 deletions packages/adapter-knex/CHANGELOG.md

This file was deleted.

77 changes: 0 additions & 77 deletions packages/adapter-knex/README.md

This file was deleted.

44 changes: 0 additions & 44 deletions packages/adapter-knex/build-test-db.sh

This file was deleted.

3 changes: 0 additions & 3 deletions packages/adapter-knex/index.js

This file was deleted.

Loading

1 comment on commit 5777fc6

@vercel
Copy link

@vercel vercel bot commented on 5777fc6 Mar 30, 2021

Choose a reason for hiding this comment

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

Please sign in to comment.