2884 commits

This branch

This branch

All branches

Author	SHA1	Message	Date
David Mehren	f9f596c0d0	Merge pull request #544 from codimd/public-api-uploads	2020-10-31 20:24:03 +01:00
Yannick Bungers	8a6e81e1c8	Merge branch 'develop' into public-api-uploads	2020-10-30 22:46:08 +01:00
Yannick Bungers	135e7ddfc2	Merge pull request #514 from codimd/note-metadata-in-db ... Save note metadata in the database	2020-10-30 22:31:37 +01:00
David Mehren	6b92b012e2	Merge pull request #512 from codimd/fix/public-api-spec	2020-10-29 22:43:28 +01:00
David Mehren	0f1cab5006	Update dependencies ... Signed-off-by: David Mehren <git@herrmehren.de>	2020-10-24 23:02:05 +02:00
David Mehren	61e6020c6b	Fix tests ... Signed-off-by: David Mehren <git@herrmehren.de> Co-authored-by: Yannick Bungers <git@innay.de>	2020-10-24 22:21:46 +02:00
David Mehren	85ee6780ad	Remove PUT /notes/{note}/metadata and corresponding service code ... Signed-off-by: David Mehren <git@herrmehren.de> Co-authored-by: Yannick Bungers <git@innay.de>	2020-10-24 22:21:45 +02:00
David Mehren	241a577a02	DB Schema: Make layout pretty ... Signed-off-by: David Mehren <git@herrmehren.de> Co-authored-by: Yannick Bungers <git@innay.de>	2020-10-24 22:21:45 +02:00
David Mehren	6a1da64cf6	Remove NoteUtils class, as the planned parsing logic is not needed anymore ... Signed-off-by: David Mehren <git@herrmehren.de> Co-authored-by: Yannick Bungers <git@innay.de>	2020-10-24 22:21:45 +02:00
David Mehren	b2085efb1d	Add missing TagRepository provider in unit tests ... Signed-off-by: David Mehren <git@herrmehren.de> Co-authored-by: Yannick Bungers <git@innay.de>	2020-10-24 22:21:44 +02:00
David Mehren	731e577158	Add E2E tests for note metadata routes ... Signed-off-by: David Mehren <git@herrmehren.de> Co-authored-by: Yannick Bungers <git@innay.de>	2020-10-24 22:21:44 +02:00
David Mehren	c1886ff1dc	NotesController: Add PUT :noteIdOrAlias/metadata route ... Signed-off-by: David Mehren <git@herrmehren.de> Co-authored-by: Yannick Bungers <git@innay.de>	2020-10-24 22:21:44 +02:00
David Mehren	3726b27849	NotesService: Implement updateNoteMetadata ... Signed-off-by: David Mehren <git@herrmehren.de> Co-authored-by: Yannick Bungers <git@innay.de>	2020-10-24 22:21:43 +02:00
David Mehren	943c8b4bab	NoteEntity: Enable eager loading and cascades for tags ... Signed-off-by: David Mehren <git@herrmehren.de> Co-authored-by: Yannick Bungers <git@innay.de>	2020-10-24 22:21:43 +02:00
David Mehren	b349d25bd7	NotesService: Get metadata from database ... Signed-off-by: David Mehren <git@herrmehren.de> Co-authored-by: Yannick Bungers <git@innay.de>	2020-10-24 22:21:43 +02:00
David Mehren	f1f57eca54	Add note metadata properties and Tag entity. ... These were planned to be parsed at runtime from the note-content in the database, but having to run a markdown parser in the backend was found to be a bad idea. Now the frontend (that already implements the parsing logic) has to set title, description and tags. Signed-off-by: David Mehren <git@herrmehren.de> Co-authored-by: Yannick Bungers <git@innay.de>	2020-10-24 22:21:42 +02:00
David Mehren	b9dfd880f7	Note.alias should be optional in db schema ... Signed-off-by: David Mehren <git@herrmehren.de> Co-authored-by: Yannick Bungers <git@innay.de>	2020-10-24 22:21:42 +02:00
David Mehren	b9279a5d20	Add note metadata to db schema ... Signed-off-by: David Mehren <git@herrmehren.de> Co-authored-by: Yannick Bungers <git@innay.de>	2020-10-24 22:21:41 +02:00
David Mehren	b7195c563c	Public API: Cleanup history schemas ... Signed-off-by: David Mehren <git@herrmehren.de> Co-authored-by: Yannick Bungers <git@innay.de>	2020-10-24 22:20:00 +02:00
David Mehren	cfe8169fc2	Public API: NoteRevisionsMetadata is not an array ... Signed-off-by: David Mehren <git@herrmehren.de> Co-authored-by: Yannick Bungers <git@innay.de>	2020-10-24 22:20:00 +02:00
David Mehren	a4ac295ada	Public API: Remove PUT /notes/{note}/metadata ... The note metadata will be automatically extracted from the note content and cannot be updated separately. Signed-off-by: David Mehren <git@herrmehren.de> Co-authored-by: Yannick Bungers <git@innay.de>	2020-10-24 22:19:59 +02:00
David Mehren	7adcad2ce6	Public API: Fix PUT /notes/{note}/permissions response ... This route should return a full `NotePermissions` object instead of only `NotePermissionsUpdate` Signed-off-by: David Mehren <git@herrmehren.de> Co-authored-by: Yannick Bungers <git@innay.de>	2020-10-24 22:19:14 +02:00
David Mehren	a640dea1aa	Public API: Update description of /notes/{note} POST and PUT routes. ... Signed-off-by: David Mehren <git@herrmehren.de> Co-authored-by: Yannick Bungers <git@innay.de>	2020-10-24 22:19:14 +02:00
David Mehren	be5b6dcf0e	NoteMetadataDto: Rename permission to permissions ... Signed-off-by: David Mehren <git@herrmehren.de> Co-authored-by: Yannick Bungers <git@innay.de>	2020-10-24 22:19:14 +02:00
David Mehren	67633c375d	Public API: Successful POST requests should result in a 201. ... Signed-off-by: David Mehren <git@herrmehren.de> Co-authored-by: Yannick Bungers <git@innay.de>	2020-10-24 22:19:13 +02:00
David Mehren	b63593aa8b	Public API: Simplify PUT /me/history/{note} ... Previously, one had to send a complete `NoteMetadata` object when updating the pinned status of a note. A new `HistoryUpdateObject` type was introduced, that only contains the pinned status boolean. Signed-off-by: David Mehren <git@herrmehren.de> Co-authored-by: Yannick Bungers <git@innay.de>	2020-10-24 22:19:13 +02:00
David Mehren	2be1defd7a	Public API: Split-out a /notes/{note}/permissions route to set only permissions ... Previously, the metadata route was used to both update note metadata (like title, description and tags) and the permissions. Additionally, one had to send many unchangeable properties. In this commit, a /notes/{note}/permissions route is introduced, that only changes permissions. Additionally, PUT /notes/{note}/metadata now needs only the properties that are actually changeable. Signed-off-by: David Mehren <git@herrmehren.de> Co-authored-by: Yannick Bungers <git@innay.de>	2020-10-24 22:19:13 +02:00
David Mehren	61014f1bc4	Update NotePermissionsUpdate DTOs to be aware of groups ... The NotePermissionsUpdateDto was not updated when group permissions were introduced. Signed-off-by: David Mehren <git@herrmehren.de> Co-authored-by: Yannick Bungers <git@innay.de>	2020-10-24 22:19:12 +02:00
David Mehren	4a975dcca2	Public API: Add routes to get uploads by user or note ... Co-authored-by: Yannick Bungers <git@innay.de> Signed-off-by: David Mehren <git@herrmehren.de>	2020-10-24 21:48:29 +02:00
David Mehren	23c07dc67d	Merge pull request #543 from codimd/media-controller-tests	2020-10-24 21:17:29 +02:00
David Mehren	26554f7168	Use unique sqlite file for every E2E test ... Previously, this lead to locking errors, when multiple test runners accessed the same database and tried to clear it or tried to insert new test data. Signed-off-by: David Mehren <git@herrmehren.de>	2020-10-24 12:30:23 +02:00
David Mehren	d42bc83e38	FilesystemBackend: Ensure uploads directory exists ... Signed-off-by: David Mehren <git@herrmehren.de>	2020-10-24 12:28:52 +02:00
David Mehren	4f3ef414d4	Add E2E tests for the /media route ... Signed-off-by: David Mehren <git@herrmehren.de>	2020-10-24 11:50:45 +02:00
David Mehren	5030a6d814	AppModule: Remove unused imports ... Signed-off-by: David Mehren <git@herrmehren.de>	2020-10-24 11:49:19 +02:00
David Mehren	0711dbb6ff	MediaService: Simplify saveFile signature ... As the `saveFile` method only really uses the files `Buffer`, this commit changes the signature so it directly gets a `Buffer` instead of a complicated `MulterFile` object. This also simplifies testing. Signed-off-by: David Mehren <git@herrmehren.de>	2020-10-24 11:49:01 +02:00
David Mehren	9aa2a64a53	UserEntity: Fix column types for create/update dates ... Signed-off-by: David Mehren <git@herrmehren.de>	2020-10-24 11:34:49 +02:00
David Mehren	5f13c34a07	UsersService: Improve logging in getNoteByIdOrAlias ... Signed-off-by: David Mehren <git@herrmehren.de>	2020-10-24 11:34:16 +02:00
David Mehren	560efc71d8	Use useStaticAssets instead of @nestjs/serve-static ... `serve-static` does not work with `createTestingModule` and is not recommended when "just" serving a few images. See https://github.com/nestjs/serve-static/issues/240 Signed-off-by: David Mehren <git@herrmehren.de>	2020-10-24 11:32:23 +02:00
David Mehren	e2696e647b	Merge pull request #534 from codimd/media-controller	2020-10-19 21:07:13 +02:00
David Mehren	a728866ebb	Public API: Add media deletion ... Signed-off-by: David Mehren <git@herrmehren.de>	2020-10-17 21:55:05 +02:00
David Mehren	3686685f08	MediaController: Add DELETE /{filename} route ... Signed-off-by: David Mehren <git@herrmehren.de>	2020-10-17 21:54:44 +02:00
David Mehren	9e7e15a20a	MediaService: Implement delete feature ... Signed-off-by: David Mehren <git@herrmehren.de>	2020-10-17 21:54:08 +02:00
David Mehren	db869418d4	FilesystemBackend: ESLint fixes ... Signed-off-by: David Mehren <git@herrmehren.de>	2020-10-17 21:53:34 +02:00
David Mehren	6e6ab84391	UsersService: Wait for the DB to find a user ... Signed-off-by: David Mehren <git@herrmehren.de>	2020-10-17 21:52:59 +02:00
David Mehren	16b5f3a5c8	Use POST /media for file upload ... The old `/media/upload` subpath does not follow the convention of REST APIs. Signed-off-by: David Mehren <git@herrmehren.de>	2020-10-17 20:58:10 +02:00
David Mehren	e0f8031fab	Public API: Update /media/upload route with supported content-types ... Signed-off-by: David Mehren <git@herrmehren.de>	2020-10-17 20:55:48 +02:00
David Mehren	ffef4425f5	MediaService: Only allow upload of common image formats and PDFs ... Signed-off-by: David Mehren <git@herrmehren.de>	2020-10-17 20:50:20 +02:00
David Mehren	ed142815e3	Add various missing imports and provider ovverides in unit tests ... Signed-off-by: David Mehren <git@herrmehren.de>	2020-10-17 20:24:07 +02:00
David Mehren	4cd80a3212	MediaController: Handle errors when trying to save file ... Signed-off-by: David Mehren <git@herrmehren.de>	2020-10-17 20:24:07 +02:00
David Mehren	8e662167dc	MediaService: Improve error handling and logging ... Add debug logging to `saveFile` method and throw the proper errors when problems with the mime type are encountered Signed-off-by: David Mehren <git@herrmehren.de>	2020-10-17 20:24:07 +02:00