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.

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

test #7

Open
wants to merge 75 commits into
base: feat/main-product-carousel
Choose a base branch
from
Open

test #7

wants to merge 75 commits into from

Conversation

michaelparkadze
Copy link
Owner

test

login and register base routes
Updated the register route and added input validation with joi.
After some research on angular and handling server errors I rather have 'message' key instead of 'error' because of the better readability.
A service that makes it easy to make api requests, currently supports [get, post, put].
Updated the login route and added input validation with joi.
- Restructured login and register into auth module
- Created auth service to handle auth related operations
- Created interceptor service to add auth token in request header
- Created guard service to handle protected routes
- Created token storage service to handle session storage related operations

*Future refactor incoming...
Improved readability, dynamically change displayed right CTA [login, profile] (Temporary) until a dropdown menu will be made to display all options
In order to develop a UI that looks good faster and some more complex components.
In order to change email in addition to the full name, the query had to be updated to be based on the id and not email.
Return an object instead of an array containing the object.
Created the layout for product page with the api call to get all the products data to showcase. Added swiper for the photo gallery of the product images.
Replaced the mock data with the actual data of the products and price, and added the remove functionality.
Changed the trigger of the dropdown to be click instead of hover.
Removed unnecessary nav on desktop and added a menu icon button instead of the placeholder button for the side menu
Organized it a bit better, added a close icon option, logout button when user is logged in and added a few nice hover animations.
michaelparkadze and others added 30 commits April 19, 2021 17:37
Created a 3 step form to checkout for user to add his billing and payment and purchase his goods. no validation what so ever was added to the forms in the steps.
Created the get orders route that returns all the orders for a specific user.
I added a street name column
It seemed to be missing a name
Added a street_name column to sql_dump.sql
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants