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

Avatar feature #782

Merged
merged 30 commits into from
Aug 31, 2024
Merged

Avatar feature #782

merged 30 commits into from
Aug 31, 2024

Conversation

dpgraham4401
Copy link
Member

Description

Implements a feature to upload and display a user avatar. Not mission critical, but good to learn.

Issue ticket number and link

Checklist

  • I have added tests that prove my fix is effective or that my feature works
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings

@dpgraham4401
add type hints for model using foreign keys to settings.AUTH_USER_MOD…
3127d6a 
…EL to use

django.contrib.auth.models.User
@dpgraham4401
update settings and urlconfig to serve media in development
140118c
@dpgraham4401
add Pillow dependency, and add ImageField to Profile model
0ea0f0f
@dpgraham4401
move profile serializers from package to module
9e9b406
@dpgraham4401
add 'media/' to path prefix for traefik reverse proxy
aba1026
@dpgraham4401
add media directory to gitignore
d917ac2
@dpgraham4401
add avatar image to profile serializer
1f75662
@dpgraham4401
apply user avatar to top navigation bar
ce0db53
@dpgraham4401
add TrakUserSerializer to ProfileSerializer so the users details are …
3acdd01 
…inlcuded with the profile views
@dpgraham4401
initial AvatarForm (no tests, saving work to stop) which will separte…
ebac16b 
… it from the UserForm and allow us to have the user upload photos
@dpgraham4401
change expected type for ProfileSlice to inlcude use information
aa70de9
@dpgraham4401
move avatar to separate component, outside the user info form
f6f4232
@dpgraham4401
somehow I needed to add the eslint.config.js file back... IDK what's …
e89aa56 
…going on here
@dpgraham4401
rename profile endpoint to my-profile endpoint, we will reimplement p…
3566488 
…rofile endpoint, however this is an easy way to retrieve the current logged in user's profile info
@dpgraham4401
add profile model viewset
62d84d0
@dpgraham4401
profile view uses username as parameter instead of ID... probably a b…
66e7978 
…ad idea
@dpgraham4401
Revert "profile view uses username as parameter instead of ID... prob…
1988a74 
…ably a bad idea"

This reverts commit 66e7978.
@dpgraham4401
saving work, marking unit test as skip for the time being, I cannot g…
6bd326c 
…et this test to work
@dpgraham4401
add Form from shadcn to component library
7006b95
@dpgraham4401
add input component
9c75045
@dpgraham4401
initial avatar POC with preview on upload
f9ed32e
@dpgraham4401
wrap our hidden input and avatar display/preview in a button that set…
9e4b4f5 
…s the fileref
@dpgraham4401
simplify avatar form
6b34313
@dpgraham4401
add updateProfile endpoint to RTK query
5473011
@dpgraham4401
saving work, POC showing optimistic preview of file uploaded as avata…
bb02255 
…r in UI
@dpgraham4401
implement simplest exmaple of react hook form and using uncontrolled …
6d32185 
…Input to capture file for upload
@dpgraham4401
POC uploading file to server
96008e1
@dpgraham4401
avatar form uses the icon as a button and updates the state of the av…
d687154 
…atar on success request
@dpgraham4401
avatar form submits the file on change instead of having to click a s…
a3af05c 
…ubmit button
@dpgraham4401
fix avatar form spec file
af7b3b7
@github-actions github-actions bot added client Related to front end workings (React/Redux) dependencies project dependency problems or modifications server Related to backend workings (Django/Rest API) test Unit tests are, or need to be, included labels Aug 31, 2024
@dpgraham4401 dpgraham4401 merged commit b14030b into USEPA:main Aug 31, 2024
11 checks passed
@dpgraham4401 dpgraham4401 deleted the avatar_feature branch August 31, 2024 03:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
client Related to front end workings (React/Redux) dependencies project dependency problems or modifications server Related to backend workings (Django/Rest API) test Unit tests are, or need to be, included
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@dpgraham4401