generated from BloomTech-Labs/template-be
-
Notifications
You must be signed in to change notification settings - Fork 2
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
Filter albums #21
Draft
MorganPeterson1313
wants to merge
28
commits into
staging
Choose a base branch
from
filterAlbums
base: staging
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Filter albums #21
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Configured Sentry.io * Added an integration test for /users * Updated controllers * Created tests for createUser model * Added tests for user models * Updated models tests * Updated /users/register, updated docs, and added tests * Added createUser and retrieveUserBy models and added tests * Added password hashing * Updated retrieveUserBy model * Added JWT to /users/register * Added /users/:user_id endpoint and added tests * Updated user endpoints and API docs * Added endpoint for logging in * Created updateUser model and added tests * Created edit user endpoint and added tests * basic photo uploading * Created remove user endpoint and added tests * Added a test for deleteUserById model * Added route protection/permissions with JWT and added endpoint tests * Created createAlbum model and added tests * Created add meta data endpoint * Added tests for add meta endpoint * Added endpoint for editing an album and added tests * Created remove album endpoint and added tests
MorganPeterson1313
requested review from
arturolei,
Brodt7258,
adrianadames and
belzy
December 10, 2019 06:07
* Configured Sentry.io * Updated file structure * Updated file structure * Added an integration test for /users * Updated controllers * Updated models export file * Added tests for users endpoint * Updated travis ci env vars * Update * Updated docs for /users/register and updated tests * Updated /users/register docs and added tests * Created tests for createUser model * Added tests for user models * Updated models tests * Updated /users/register, updated docs, and added tests * Added createUser and retrieveUserBy models and added tests * Added password hashing * Updated retrieveUserBy model * Added JWT to /users/register * Updated app.json with jwt key for review apps * Updated .travis.yml with jwt key for testing * Test * Test * Test * Added error handler for user routes * Added error handler for user routes * Updated .travis.yml * Added /users/:user_id endpoint and added tests * Updated user endpoints and API docs * Added endpoint for logging in * Added endpoint for logging in * Updated API Docs * Updated API Docs * Udpated API docs and updated endpoint integration tests * Refactored tests * Refactored controller integration tests * Created updateUser model and added tests * Created edit user endpoint and added tests * basic photo uploading * Update knexfile.js * Update knexfile.js * Created remove user endpoint and added tests * Updated API docs * further cleaned up merge * Added a test for deleteUserById model * Added route protection/permissions with JWT and added endpoint tests * Testing CORS options * Testing CORS options * Testing heroku timeout * Testing heroku timeout * Testing heroku timeout * Testing heroku timeout * Testing heroku timeout * Testing heroku timeout * Testing heroku timeout * Testing heroku timeout * Testing heroku timeout * Testing heroku timeout * Fixed a bug * Fixed a bug * Testing cors * Testing cors * Testing cors * Testing cors * Testing cors * Testing cors * Testing cors * Testing cors * Testing cors * Testing cors * Testing cors * Testing cors * Testing cors * Testing cors * Testing cors * Testing cors * Testing cors * Testing cors * Configured CORS for all environments * Configured CORS for all environments * Configured CORS for all environments * Configured CORS for all environments * Updated API docs * Updated API docs * Updated API docs * Refactored controllers * Created createAlbum model and added tests * Created add meta data endpoint * Added tests for add meta endpoint * Added endpoint for editing an album and added tests * Created remove album endpoint and added tests * Created models and validation functions * Resolving merge conflicts * Updated endpoint tests * Updated API docs and added tests * Endpoint adds media to albums * Endpoint adds media to albums * Created endpoint for uploading media and added tests * Updated create media endpoint * Created endpoint for uploading media and added tests * Added permission checking and tests for create media endpoint * Added permission checking and tests for create media endpoint * Added retrieveAlbumsMedia model and tests * Created endpoint for getting an albums media and added tests * Updated get albums media route description * Updated get albums media route description * Updated get albums media route description * Added cover_url and thumbnail_url * Updated album endpoints with cover_url * Added get users media endpoint and tests * refactored media upload * added parsing IDs to integer * testing cloudinary * Added absolute path for tmp media folder * Added .getkeep * Updated utils * Added bigSerial to user_id for postgresql * Refactored migrations
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Fixes # (issue)
Type of change
Please delete options that are not relevant.
Change Status
How Has This Been Tested?
Checklist