-
Notifications
You must be signed in to change notification settings - Fork 0
Initial Release #43
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.
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
base: main
Are you sure you want to change the base?
Initial Release #43
Conversation
#1) * Added initial configuration for poetry and the venv. Also added the .gitignore and updated the README with instructions to use poetry. * Added everyone on the team as authors in the pyproject.toml * Added default Python .gitignore * Added .DS_Store
* branches * added app * added complete skeleton arhcitecture for eg_app, which is our backend. I have added static js and style sheet files. * small changes * small * added poetry files, tested basic index.html showing up * added revised git ignore * cleanup * revised git ignore * Delete .DS_Store * Delete evergreen/eg_app/__pycache__ directory * Delete evergreen/db.sqlite3 * Delete evergreen/eg_app/.DS_Store * Delete evergreen/eg_app/static/.DS_Store * Delete evergreen/eg_app/templates/.DS_Store * Delete evergreen/eg_app/migrations/__pycache__ directory * Delete gitignore --------- Co-authored-by: Sumanyu Janapareddy <sumanyujanapareddy@Sumanyus-MacBook-Pro-2.local> Co-authored-by: Eric Butcher <107886303+Eric-Butcher@users.noreply.github.com>
* added index.html & CSS * added changes to HTML and CSS made Project-Evergreen consistent, removed other projects and changed email link to discord * Project-Evergreen -> Project Evergreen
* added terminal help codes * docker file made * dockerfile and docker-compose.yml created and tested * updated contents from dev branch * Docker files changed to fix directory issues
* added index.html & CSS * Add some real simple JavaScript to satisfy P1 Obj1 * added changes to HTML and CSS made Project-Evergreen consistent, removed other projects and changed email link to discord * Project-Evergreen -> Project Evergreen --------- Co-authored-by: rancescolon <109361303+rancescolon@users.noreply.github.com>
…fully we can move on to objective 2. (theoretically) (#11) * added todo to DockerFile * updated gitignore to cover additional pesky files from IDEs like Pycharm (generated from toptal) * changed settings.py to just BASE_DIR instead of looking at templates * added to views.py 2 functions to render css and js files. Also moved function.js and styles.css to different paths in the static folder, as per reqs * changed index.html to have django understand its static files and added image * cleaning * Delete .idea directory * changed file structure, added whitenoise, change settings.py * some fioxes * removed excess static files
* added todo to DockerFile * updated gitignore to cover additional pesky files from IDEs like Pycharm (generated from toptal) * changed settings.py to just BASE_DIR instead of looking at templates * added to views.py 2 functions to render css and js files. Also moved function.js and styles.css to different paths in the static folder, as per reqs * changed index.html to have django understand its static files and added image * cleaning * Delete .idea directory * changed file structure, added whitenoise, change settings.py * some fioxes * removed excess static files * implemented Gia's changes for docker file into here * added login html and css --------- Co-authored-by: giagermain <99298965+giagermain@users.noreply.github.com>
…password encryption method in util, and SKELETON handler for login and reg (#27) * added onclick property to html form for passwords to connect to password validator, created function (js) inline to changer color of url bar depending on if the password validator meets * added pattern for password validator in urls.py and function to plumb together Alex's method in views.py * working commit for checking if the password is validated * added encrypt function * changed a few wordings * fixed issues with seeing validation text
* Added poetry to docker (merging Gia's poetry docker stuff) (#14) * added todo to DockerFile * updated gitignore to cover additional pesky files from IDEs like Pycharm (generated from toptal) * changed settings.py to just BASE_DIR instead of looking at templates * added to views.py 2 functions to render css and js files. Also moved function.js and styles.css to different paths in the static folder, as per reqs * changed index.html to have django understand its static files and added image * cleaning * Delete .idea directory * changed file structure, added whitenoise, change settings.py * some fioxes * removed excess static files * implemented Gia's changes for docker file into here * added login html and css * Create email & password validators w/ tests (#12) * removed req.txt and added email-validator to poetry (#15) * Replaced docker configuratino for a mongoDB with postgresql * Updated settings.py for django with the setting for the postgres db * Updated poetry configuration to get rid of mymongo and include the pip library necessary for django to communicate with postgres which is psycopg version 3 * Amended docker-compose.yml so the database container will close when docker is shut down * Made changes that seems to get this to work * Fixed wait script * Got the server running fine. * Fixed flow for setting up the docker containers and migrating * Mentioned that poetry needs to be used from evergreen directory * env.example * fixed comments about insecurity in settings.py --------- Co-authored-by: Sumanyu Janapareddy <111717258+sumanyuj@users.noreply.github.com> Co-authored-by: Alex M <weirdalex0302+github@gmail.com>
* Account models and signup/login code * Adding routing for auth wip * fix wrong name for pwd conf * Refactor to use Django auth, add logout * remove now-unnecessary models * drop debug prints * Address feedback - Refactor register dupe check to avoid try/catch - Add explicit kwargs to `create_user` call in `register` - Make root path a global var for all `views` methods
* README now reflects migratinos happening during container build * Removed mention of makemigrations
* Add files via upload * added file handler to view.py * Delete views.py * Update views.py * Update views.py * Update views.py * Update views.py * Update views.py * Update views.py * Update views.py S -> s * Update views.py
* logout button and login home page added * reformatted index.html * logout and login included to display whether a user is logged in or not * fixed if user * resolved alex's comments
* Added poetry to docker (merging Gia's poetry docker stuff) (#14) * added todo to DockerFile * updated gitignore to cover additional pesky files from IDEs like Pycharm (generated from toptal) * changed settings.py to just BASE_DIR instead of looking at templates * added to views.py 2 functions to render css and js files. Also moved function.js and styles.css to different paths in the static folder, as per reqs * changed index.html to have django understand its static files and added image * cleaning * Delete .idea directory * changed file structure, added whitenoise, change settings.py * some fixes * removed excess static files * implemented Gia's changes for docker file into here * added login html and css * Create email & password validators w/ tests (#12) * removed req.txt and added email-validator to poetry * Reset all files the same as dev * Use as source code for chat functionality * LF --------- Co-authored-by: Sumanyu Janapareddy <111717258+sumanyuj@users.noreply.github.com> Co-authored-by: Alex M <weirdalex0302+github@gmail.com>
…CURSED FRONTEND LANGUAGE KNOWN AS JAVASCRIPT. THIS IS MY PLEA TO USE REACT. I BEG OF YOU. (#40) * Added poetry to docker (merging Gia's poetry docker stuff) (#14) * added todo to DockerFile * updated gitignore to cover additional pesky files from IDEs like Pycharm (generated from toptal) * changed settings.py to just BASE_DIR instead of looking at templates * added to views.py 2 functions to render css and js files. Also moved function.js and styles.css to different paths in the static folder, as per reqs * changed index.html to have django understand its static files and added image * cleaning * Delete .idea directory * changed file structure, added whitenoise, change settings.py * some fioxes * removed excess static files * implemented Gia's changes for docker file into here * added login html and css * Create email & password validators w/ tests (#12) * removed req.txt and added email-validator to poetry * Reset all files the same as dev * DO NOT MERGE: Use as source code for chat functionality * correct incorrect tags in index.html * updated updateFeed function so that it loops through posts * added MEDIA ROOT paths for settings.py and path for it in urls * updated updateFeed() function * minor changes * q q * fixed html login and reg duplicates and button/input layouts * deleted dislike * removed req.txt * remove migrations folder * Delete evergreen/eg_app/migrations directory actually delete migrate * fixed all the weird merge stuff * removed duplicate in manage.py * removed newline * changed dockerFile * fixed small changes in function.js * changed from CRLF to LF(among other things) * added back some paths (sorry!) * added back init.py and added gitignore * fixed encode * fixed whiteline * added newline to caption * changes to stuff * users can see their emails on posts now * fixed weird changes to function.js * semi colon and some changes to TYPES * removed duplicate delete * taked out type ignore * some unresolved stuff * some changes * some changes * some stuff * changed user string * added conditionality to just post the caption * added html.escape * added soem changes to like method * latest changes * reverted to not finite likes * added Alex's bugfix for views.py likes among other changes. * readded userLikes in models.py --------- Co-authored-by: Alex M <weirdalex0302+github@gmail.com> Co-authored-by: Gianna Germain <ggermain@buffalo.edu>
sumanyuj
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!!!
|
There are a bunch of problems with this code currently. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CodeQL found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.
This resolves 136 of 140 flagged issues
* Updated READMe * Added gh flavored markdown stuff for Alex as well as a link to wSL documentation so that doesn't confuse people who are not familiar. * Added changes
* Added autopep8 formatting * isort configuration * Added local formatting options for python, JS, & HTML * included extensions.json
Fixed Dependabot's concerns and all but one CodeQL problem I left the superfluous argument in HTML because deletePost() is a rats nest of incompleteness that I am NOT going to get into right now
…#47) Got GitHub actions working for testing and formatting.
* trasnfering image upload changes to my linux * - 404 error now 200 OK - load image in new tab - fixed mix matched "upload_to" * to linux * linux pt 2 * to linux * rebasing * actual updated branch * Fix docker/nginx config for static files This is only for static files, idk exactly where uploads are going so I'll leave that to you * images work * Debug False * fixed bot issues * formatted * images work * Fixed erics issues * resolved new issues * issues fixed pt2 --------- Co-authored-by: Alex M <apmchugh@buffalo.edu>
* adding channels to poetry * added middleware and layer to settings.py * created consumers.py to handle web sockets * routing for web sockets * updated asgi to handle web sockets in django * working websockets * minor fixes * converted like button to websockets * run make some fixes * fixed exception handler * removed unused imports * ran make * changed * fixed black * commented out cmd * removed comment * Update evergreen/eg_app/consumers.py Co-authored-by: Alex M <weirdalex0302+github@gmail.com> * fixed issue with web sockets * ran make * Update urls.py * Update functions.js * removed allow all orgins * ran make * removed duplicate like button * updated poetry lock * increase static file limit nginx * passing in the self signed cert files * updated front end logic and visual for updated like button and uploading content via sockets * updated backend to handle websockets handling content, like button logic, and more! * running make * poetry lock fix * added toml * Update evergreen/staticfiles/js/functions.js Co-authored-by: Alex M <weirdalex0302+github@gmail.com> * removed blank lines * fixed exception * minor image access check * better file type checking, making sure to actually check if image is uplaoded, magic numbers, and a bit more! * cleanup logs * removed comment * fixed pesky braket issue (red alert) * rules of engagement * ran make * go rid of styles * quick fix to limit text size and image size on backend (and alert user) * f-strings baby * ran magic make * removed ssl stuff and updated rules of gentlemanly plant engagement * removed 8MB limit that was non working * type annoation * restored nginx ignore --------- Co-authored-by: Alex M <weirdalex0302+github@gmail.com>
Just had to add "http://localhost" to Django CSRF_TRUSTED_ORIGINS. That's it.
* fixed escaping * added redirects note and url to read me * i cant spell --------- Co-authored-by: Sumanyu Janapareddy <111717258+sumanyuj@users.noreply.github.com>
Honestly idk why we didn't have this already
* updated readme for env * Update README.md
* catches frame in recieve method to check for if frame size is exceeded or not. * added csrf exempt to logout view!! * ran make
* fixed nontransparent background * added redirects
* tmp basic njs starter pending piazza ans * nginx rate limiting for demo this is totally incompatible with the AO, but should hopefully help keep us online in the demo :fingers_crossed: :fingers_crossed: :fingers_crossed: * demo ip
* Added ignores so my IDE doesn't yell at me. * Creared relay_handler model. * Fixed relay_request to use the request.POST variable instead. * Added command to create new RelayDevic. * Returns an error message if the relay handler failed to add timelapse to psotgres. * Fixed exception handling in relay_processor.py. * Reigster models in admin file. * URL for the timelapse. * Fixing linter and type-checking erroring. * Time lapse now has lcoation on the main page. * Added relay upload path. * Made some fixes * Fixed large files being 413'd. * Linting with make to fix bugs and error codes from flake8 about unused imports. Also added max_length to models.
* fixed escaping * added redirects note and url to read me * i cant spell * fixed readme * small typo * fixed clunkiness --------- Co-authored-by: Sumanyu Janapareddy <111717258+sumanyuj@users.noreply.github.com>
Added README testing procedure for Part3
Weren't using that anyway, just a leftover from some prior stuff
…ogin. (#52) Added a check with flake8 to check the cyclomatic complexity of code at level 10. Simplified logic for registration and login. Added testing logic for validation functionality. --------- Co-authored-by: Alex M <weirdalex0302+github@gmail.com> Co-authored-by: Sumanyu Janapareddy <111717258+sumanyuj@users.noreply.github.com> Co-authored-by: giagermain <99298965+giagermain@users.noreply.github.com>
Adding pep8 enforcement of naming.
Bumps [django](https://github.com/django/django) from 5.1.3 to 5.1.5. - [Commits](django/django@5.1.3...5.1.5) --- updated-dependencies: - dependency-name: django dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Fixed the settings to be secure in the settings.py and .env files.
The final step: Merging it all to
main!Make sure not to squash merge this one
Finishes #37