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

Frontend tagging & search #44

Merged
merged 67 commits into from
Aug 25, 2023
Merged

Frontend tagging & search #44

merged 67 commits into from
Aug 25, 2023

Conversation

skoriop
Copy link
Member

@skoriop skoriop commented Aug 5, 2023 •
edited
Loading

  • Please check if the PR fulfills these requirements
  • The commit message follows our guidelines
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)
  • The code runs on the contributors machine.
  • The code follows the code style guide of this project.

  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
    Feature

  • What is the current behavior? (You can also link to an open issue here)
    No endpoint to tag images

  • What is the new behavior (if this is a feature change)?

  • Added DopyImage, Event and Tag models (along with appropriate relations).
  • Added image tagging endpoint
  • Added user roles (for now only way to change is direct database calls, should add to admin dashboard later)
  • Fixed Make oauth accept only emails of bits domain #29
  • Modify image upload endpoint for DopyImage
  • Add tag update endpoint (for normal users)
  • Integrate tag update endpoint with frontend
  • Add search images endpoint
  • Integrate search images endpoint with frontend

  • Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

  • Other information:

@skoriop skoriop changed the title Frontend tagging Frontend tagging [WIP] Aug 5, 2023
pnicto and others added 17 commits August 5, 2023 23:05
@pnicto
build: expose db port, remove unnecessary port, change bind mount dir
ed82dbf
@pnicto
build: remove line erroring the build process as the file does not exist
0a7b8d8
@pnicto
build: create uploads directory and give necessary perms
58b87cd
@pnicto
build: change exposed ports for nextauth in prod, simplify bind mount…
53065fd 
… for uploads, expose db port for migrations
@pnicto
docs: add info about running the prod image
82d3dad
@skoriop
feat: modfied profile page for dopy accounts
f4fab59
@skoriop
feat: modify image upload for dopy
e4642c0
@skoriop
refactor: remove userData variable
cb58c89
@skoriop
refactor: move login page to pages folder
7308bc0
@skoriop
refactor: rename db table names
7e2df15
@skoriop
feat: add update tags endpoint
bc45b35
@skoriop
Revert "feat: modify image upload for dopy"
27b2d66 
This reverts commit e4642c0.
@skoriop
Revert "feat: modfied profile page for dopy accounts"
88ad7ba 
This reverts commit f4fab59.
@skoriop
fix: update tag model in prisma
0c9608b
@skoriop
fix: change updateTags endpoint to use tag values
f4f078d 
seems like the best way to avoid frontend spaghetti
@skoriop
feat: connect frontend to update tags endpoint
7c26f24 
also pass image id to modal component
@skoriop
fix: make case of tag value uniform
74a6f9a
@skoriop skoriop changed the title Frontend tagging Frontend tagging & search Aug 19, 2023
pnicto and others added 22 commits August 20, 2023 09:06
@pnicto
fix: modal image previews on other browsers that are not firefox
62e1ffb 
memory management in firefox is different from the rest
@pnicto
Revert "refactor: add event year as part of label"
0668751 
This reverts commit 27bf0b8.
@pnicto
feat: create event year table
f326f8a
@pnicto
feat: finish endpoint to get available events for a given year
3cc9d6f
@pnicto
feat: finish request to get available events for a given year for the…
c72c227 
… multiselect
@pnicto
refactor: change the flow to show event year selection first
d5ea2e2
@pnicto
feat: fetch events button to make the fetch available events request
08a9afb
@pnicto
refactor: disable multiselect if there are no events
f47638d
@pnicto
refactor: remove event year as only event id is used for finding images
e14a978
@pnicto
test: add more events with years
f9d4529
@pnicto
style: fix formatting with prettier
97b3f4b
@pnicto
refactor: change the req from post to get
6179cba 
semantically(god knows what these are) makes more sense
@pnicto
test: add random tags to images
7c8f72d
@pnicto
fix: return tags along with images
0e0f25d 
I forgor 💀 to include tags
@pnicto
Revert "feat: fetch events button to make the fetch available events …
6719710 
…request"

This reverts commit 08a9afb.
@pnicto
feat: fetch events on year change instead of an extra button click
bebf5eb 
the extra button was supposed to be pressed every time they change the year which makes it confusing to use
@pnicto
fix: reset selected events when the new available events are fetched
1ecbc9c
@pnicto
feat: add regex validation for uid input field
95aa80c
@skoriop
test: reset db before adding test data
04bd1b5
@skoriop
fix: pass tags to image modal
c1ee8b6
@skoriop
refactor: use filter instead of for...of
91823b8
@skoriop
refactor: use 'add tag' instead of 'create tag'
5f36019
@pnicto pnicto merged commit e4572fe into master Aug 25, 2023
1 check passed
@pnicto pnicto deleted the frontend-tagging branch August 25, 2023 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pnicto pnicto pnicto approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Make oauth accept only emails of bits domain
2 participants
@skoriop @pnicto