feature/tagging #16

cameron · 2021-10-12T01:50:00Z

cameron commented

2021-10-12 01:50:00 +00:00

Add endpoints for tagging files

cameron added 6 commits 2021-10-12 01:50:02 +00:00

Update dependencies 51081d01c6

Create Tag tables and Add Tag endpoint 4834cacfc3

Improve add tag endpoint and add get tag endpoint cf9dd826c1

Flattened out the add tag logic to make it more functional.

Add created timestamps for tags 50d557001b

Elevate insertion logs to info and fix error logs 125ba6192e

Update dependencies

Core Repos/ImageApi/pipeline/pr-master Something is wrong with the build of this commit

Details

39d284dbbb

cameron force-pushed feature/tagging from 39d284dbbb to 14ab02a1ec

2021-10-12 01:52:35 +00:00

Compare

cameron added 1 commit 2021-12-01 01:08:27 +00:00

Create Delete Tag endpoint

Core Repos/ImageApi/pipeline/pr-master There was a failure building this commit

Details

9cd19d03eb

cameron added 2 commits 2022-03-18 02:07:58 +00:00

Merge branch 'master' into feature/tagging c8cae28c9f

Move tags to their own module

Core Repos/ImageApi/pipeline/pr-master This commit looks good

Details

40c79d13db

cameron added 2 commits 2023-03-19 01:00:55 +00:00

Update and Migrate Diesel to 2.0 68bfcbf85f

Almost have tag support working, still figuring out how to get photo
tags.

Fix tags all and tags for photo endpoints f98d3261a7

cameron added 1 commit 2023-03-19 16:13:30 +00:00

Add Endpoint for Batch add/removal of tags on a file 8bcd9440bf

cameron added 1 commit 2023-03-19 16:29:47 +00:00

Fix tests fbcfc68e01

cameron added 1 commit 2023-03-19 19:07:00 +00:00

Refactor tags services and added tests de4041bd17

Implemented some functionality which will allow the service
configuration of the app to be split among the features for readability
and testability.

cameron added 5 commits 2023-03-22 19:30:37 +00:00

Add filtering by Tag to /images endpoint 7d05c9b8bf

Fix tagging file with new tag 02444de7fa

When tagging a file with a brand new tag, we were using the number of
rows affected as the ID instead of doing the query for the ID of the row
we just inserted, this should fix when we tag a photo with a new tag.

Fix searching by tags and fixed All FilterMode 5a2dce85e8

Manually parsing the tag_ids for the file filtering isn't amazing, but
this works in a more friendly format.

Also the All filter mode was set up in the wrong direction instead of
checking that the file had ALL the tag ids provided, it checked that all
the tag-ids were on a file, which is too restrictive and wouldn't show
many files. Perhaps an ONLY option could exist for being even more
specific.

Extract FileSystem to a trait for better testability 4ded708911

Added some tests around filtering and searching by Tags. Added the
ability to use an in-memory Sqlite DB for more integration tests.

Fix clippy warnings ae8665f632

cameron reviewed 2023-03-22 20:35:52 +00:00

cameron left a comment

LGTM

cameron added 2 commits 2023-03-26 00:53:31 +00:00

Fix warnings, and general code cleanup ae2642a544

Add the ability to query files by tags independent of file path a9109fc52d

cameron added 1 commit 2023-03-26 00:59:43 +00:00

Run clippy 7863990c68

cameron added 1 commit 2023-04-03 12:42:08 +00:00

Fix file upload 8bcd837038

Add a flag for new files so we can skip the exists check when seeing if
the new file is within the base directory.

cameron merged commit 5228cb14e0 into master

2023-04-10 12:55:28 +00:00

cameron deleted branch feature/tagging

2023-04-10 12:55:29 +00:00

cameron referenced this issue from a commit

2023-04-10 12:55:29 +00:00

Merge pull request 'feature/tagging' (#16) from feature/tagging into master

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: Apps/ImageApi#16