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

Feedback #1

Open
wants to merge 132 commits into
base: feedback
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
132 commits
Select commit Hold shift + click to select a range
aeec86a
Setting up GitHub Classroom Feedback
github-classroom[bot] Sep 19, 2022
f90b798
feat: initialize frontend project with create-react-app
leximainw Sep 20, 2022
3c1c3fc
feat: implement Login component
leximainw Sep 21, 2022
dd78773
fix: make login password label uppercase
leximainw Sep 21, 2022
bf781b4
feat: handle user registration in Login component
leximainw Sep 21, 2022
d04fcac
feat: add submit button to Login component
leximainw Sep 21, 2022
1630fd8
build: add Axios
leximainw Sep 21, 2022
2049d6d
feat: implement Login post
leximainw Sep 21, 2022
9f738fc
refactor: remove pair programming comment
leximainw Sep 21, 2022
6ddb3a0
fix: don't post to login URL when registering
leximainw Sep 21, 2022
c3820ea
feat: add register endpoint
leximainw Sep 21, 2022
89addb3
feat: add post to registration endpoint in Login
leximainw Sep 21, 2022
30f0cf6
Merge pull request #2 from Momentum-Team-14/login
leximainw Sep 21, 2022
ec09200
feat: create Card component
leximainw Sep 21, 2022
a728b2c
feat: create CardList component
leximainw Sep 21, 2022
800a56b
style: remove semicolons for consistency
leximainw Sep 21, 2022
3ec514d
feat: add mock data file
leximainw Sep 21, 2022
c9a818b
feat: display CardList
leximainw Sep 21, 2022
7820c25
fix: add pk to Card in CardList
leximainw Sep 21, 2022
bd4fae8
refactor: merge Card component into CardList
leximainw Sep 21, 2022
def91b3
feat: create CardDetails component
leximainw Sep 21, 2022
6568e83
feat: add owner name to CardDetails
leximainw Sep 21, 2022
8ac00d7
feat: add callback on selecting a card to CardList
leximainw Sep 21, 2022
bbff36c
feat: show selected card
leximainw Sep 21, 2022
9a015ab
Merge pull request #3 from Momentum-Team-14/CardDetails
leximainw Sep 21, 2022
b6b1c18
Merge pull request #4 from Momentum-Team-14/CardList
leximainw Sep 21, 2022
8090e1d
feat: change to live API
leximainw Sep 22, 2022
6cd2add
feat: add endpoint for fetching all cards
leximainw Sep 22, 2022
13aeba2
fix: target correct URLs for API card list endpoint
leximainw Sep 22, 2022
7de671b
feat: add helper function to get card list from API
leximainw Sep 22, 2022
1b5e99e
feat: add React-Router dependency
leximainw Sep 22, 2022
35125f8
feat: integrate React Router into app
leximainw Sep 23, 2022
f2998ef
feat: navigate to Card route in CardList
leximainw Sep 23, 2022
f9cca6c
feat: fetch mock card from ID in CardDetails
leximainw Sep 23, 2022
b808ab6
feat: add pk to user mock data
leximainw Sep 26, 2022
355a3cf
fix: make endpoint URLs consistent with backend API
leximainw Sep 26, 2022
f824812
feat: add getter for comments on individual card
leximainw Sep 26, 2022
03d0bbd
feat: fetch real card data from backend API
leximainw Sep 26, 2022
67a9e0a
Merge pull request #5 from Momentum-Team-14/endpoints
leximainw Sep 26, 2022
f9175c0
fix: fix typo when getting results from backend API
leximainw Sep 26, 2022
eeee615
Merge pull request #6 from Momentum-Team-14/endpoint-fix
leximainw Sep 26, 2022
86753f6
fix: add filter to get comments from a specific card from all-cards e…
leximainw Sep 26, 2022
66f9041
feat: add CommentList component
JeffBuck8 Sep 26, 2022
9c4f9b9
fix: remove comments getter since API now returns comment list
leximainw Sep 26, 2022
7ff86dd
Merge pull request #7 from Momentum-Team-14/endpoint-revert
leximainw Sep 26, 2022
a016bd4
Merge branch 'main' into comments
JeffBuck8 Sep 26, 2022
f5b5003
feat: add CommentList to CardDetails component
JeffBuck8 Sep 26, 2022
a2e1f31
Merge pull request #8 from Momentum-Team-14/comments
JeffBuck8 Sep 26, 2022
e14d5af
merge: main -> routing
leximainw Sep 26, 2022
e2f7ec1
feat: redirect homepage to card list page
leximainw Sep 26, 2022
feb362f
feat: add getCard endpoint
leximainw Sep 26, 2022
596c855
fix: fetch card from backend API if user navigates back to card detai…
leximainw Sep 26, 2022
1d877cb
Merge pull request #9 from Momentum-Team-14/routing
JeffBuck8 Sep 26, 2022
0fea9a1
fix: make auth URLS consistent with backend API
leximainw Sep 27, 2022
9c0c0da
feat: automatically log user in after registering
leximainw Sep 27, 2022
6367da9
feat: add Bootstrap 5.2.0
leximainw Sep 27, 2022
e9067ae
feat: add ignored directory for miscellaneous development files
leximainw Sep 27, 2022
ff7963b
fix: display 'Register' in Login component in registration mode
leximainw Sep 27, 2022
c3aeff0
fix: call login route after register success, not immediately
leximainw Sep 27, 2022
e8f8828
feat: display errors returned from backend API
leximainw Sep 27, 2022
2997096
Merge pull request #10 from Momentum-Team-14/login
leximainw Sep 27, 2022
3df0cee
feat: call onSuccess when auth token is received
leximainw Sep 28, 2022
5139bc1
refactor: move do-nothing arrow function into utils
leximainw Sep 28, 2022
101b742
fix: increase linter happiness
leximainw Sep 28, 2022
4bb15a2
fix: add call to success callback
leximainw Sep 28, 2022
7212064
refactor: remove obsolete TODOs
leximainw Sep 28, 2022
f2d69aa
fix: remove errors on success
leximainw Sep 28, 2022
1025004
feat: add username to Login component callback result
leximainw Sep 28, 2022
8d45e57
feat: store username and auth token in local storage
leximainw Sep 28, 2022
42b42ee
Merge pull request #11 from Momentum-Team-14/login-token
leximainw Sep 28, 2022
6355b2b
feat: add style in css
JeffBuck8 Sep 28, 2022
ed27868
feat: implement ProfileNav component
leximainw Sep 28, 2022
bfba01b
show Profile navbar at the top of all pages
leximainw Sep 28, 2022
17b6b50
feat: store login result in App
leximainw Sep 28, 2022
afdc46d
feat: call logout callback when logout option is clicked
leximainw Sep 28, 2022
6333720
feat: add logout endpoint
leximainw Sep 28, 2022
485b8b1
feat: add logout behavior in App component
leximainw Sep 28, 2022
f8f2c64
fix: correct arguments passed to useLocalStorageState
leximainw Sep 28, 2022
db677aa
feat: log user out when Logout is clicked
leximainw Sep 28, 2022
e5e1d6c
feat: call profile callback when Profile is clicked
leximainw Sep 28, 2022
140d221
feat: add styles to logged-in profile navbar
leximainw Sep 28, 2022
a879d20
feat: display username in logged-in profile navbar
leximainw Sep 28, 2022
79b484a
fix: prevent default event on logged-in profile navbar links
leximainw Sep 28, 2022
e43368f
fix: pass empty results object to logout callback
leximainw Sep 28, 2022
d727f8d
Merge pull request #12 from Momentum-Team-14/profile-nav
leximainw Sep 28, 2022
89d4726
feat: add encodeUsername to allow encoding arbitrary UTF-8 text to Dj…
leximainw Sep 29, 2022
db0d45f
fix: don't include underscore in space-ignorable characters
leximainw Sep 29, 2022
3e1c1aa
refactor: rename inner index variable to 'j' to avoid potentially-con…
leximainw Sep 29, 2022
7e92ad1
fix: use correct 6-bit mask constant
leximainw Sep 29, 2022
25930df
feat: implement decodeUsername to reverse encodeUsername
leximainw Sep 29, 2022
be36df2
feat: add login and register endpoints
leximainw Sep 29, 2022
10fa15b
refactor: use register and login endpoints in Login component
leximainw Sep 29, 2022
f4912dc
feat: trim whitespace in usernames
leximainw Sep 29, 2022
9f66dea
feat: make card switch sides on click
leximainw Sep 29, 2022
0850853
feat: apply user-customized styles to card
leximainw Sep 29, 2022
e169220
Merge pull request #13 from Momentum-Team-14/card-styles
leximainw Sep 29, 2022
71846bc
Merge pull request #14 from Momentum-Team-14/user-encode
leximainw Sep 29, 2022
a6126e3
feat: add endpoint to create a new card
leximainw Sep 29, 2022
9031bd4
fix: only create new style if is not a pk
leximainw Sep 29, 2022
9bca7d6
feat: add empty card creation page
leximainw Sep 29, 2022
a7c78ef
feat: add fields for card title, outer message, and inner message
leximainw Sep 29, 2022
7af8512
feat: add button component for styling
JeffBuck8 Sep 29, 2022
df44ed0
merge main
JeffBuck8 Sep 29, 2022
3e9f01a
fix: add default style options to dropdown component
JeffBuck8 Sep 29, 2022
4c7f506
Merge pull request #15 from Momentum-Team-14/styles
JeffBuck8 Sep 29, 2022
370eebb
chore: remove unused dependencies
JeffBuck8 Sep 29, 2022
b17126b
Merge pull request #16 from Momentum-Team-14/remove-deps
JeffBuck8 Sep 29, 2022
4cc39ac
feat: create UserProfile component
leximainw Sep 29, 2022
45dac69
merge changes from main
leximainw Sep 29, 2022
ae4b106
refactor: rename DropDownMenu to CardStyles
leximainw Sep 29, 2022
d6eda0a
style: add newline to end of App.js
leximainw Sep 29, 2022
8a6a243
refactor: rename 'src/styles.js' to 'src/CardStyles.js'
leximainw Sep 29, 2022
86c5c27
refactor: remove debug route
leximainw Sep 29, 2022
56ad8a9
build: update dependencies list
leximainw Sep 29, 2022
b463e65
feat: add CardStyles component to CardCreate
leximainw Sep 29, 2022
7295d33
refactor: create CardStyles dropdowns dynamically and track current v…
leximainw Sep 29, 2022
6834698
fix: use correct property 'border_color'
leximainw Sep 29, 2022
ab4e631
feat: add onUpdate hook to CardStyles
leximainw Sep 29, 2022
cc19e47
fix: call onUpdate in update helper function
leximainw Sep 29, 2022
d4d4c1f
docs: update README.md with directions to set up the project
leximainw Sep 29, 2022
2384be4
docs: add URL for project live deployment
leximainw Sep 30, 2022
d7295ff
Merge pull request #17 from Momentum-Team-14/create-endpoints
leximainw Sep 30, 2022
6dc5a13
fix: use current value for style in callback, instead of stale state
leximainw Sep 30, 2022
7a4653a
fix: make Font dropdown show fonts instead of colors
leximainw Sep 30, 2022
e8e0661
fix: make Text Align dropdown show text alignment instead of colors
leximainw Sep 30, 2022
7ad4a7e
fix: add createCard and createStyle routes to urls
leximainw Sep 30, 2022
a6451eb
feat: make CreateCard component submit new card
leximainw Sep 30, 2022
16f4212
feat: add redirect to new card
leximainw Sep 30, 2022
cc8428f
fix: decode username encoding in CardDetails
leximainw Sep 30, 2022
8b540f9
Merge pull request #18 from Momentum-Team-14/create-card
leximainw Sep 30, 2022
492cf59
fix: redirect all URLs on Netlify to index.html to work with React Ro…
leximainw Sep 30, 2022
1b72ee8
Merge pull request #19 from Momentum-Team-14/netlify-fix
leximainw Sep 30, 2022
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
24 changes: 24 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
@@ -0,0 +1,24 @@
# See https://help.github.com/articles/ignoring-files/ for more about ignoring files.

# dependencies
/node_modules
/.pnp
.pnp.js

# testing
/coverage

# production
/build

# misc
/.ignore
.DS_Store
.env.local
.env.development.local
.env.test.local
.env.production.local

npm-debug.log*
yarn-debug.log*
yarn-error.log*
70 changes: 70 additions & 0 deletions README-create-react-app.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,70 @@
# Getting Started with Create React App

This project was bootstrapped with [Create React App](https://github.com/facebook/create-react-app).

## Available Scripts

In the project directory, you can run:

### `npm start`

Runs the app in the development mode.\
Open [http://localhost:3000](http://localhost:3000) to view it in your browser.

The page will reload when you make changes.\
You may also see any lint errors in the console.

### `npm test`

Launches the test runner in the interactive watch mode.\
See the section about [running tests](https://facebook.github.io/create-react-app/docs/running-tests) for more information.

### `npm run build`

Builds the app for production to the `build` folder.\
It correctly bundles React in production mode and optimizes the build for the best performance.

The build is minified and the filenames include the hashes.\
Your app is ready to be deployed!

See the section about [deployment](https://facebook.github.io/create-react-app/docs/deployment) for more information.

### `npm run eject`

**Note: this is a one-way operation. Once you `eject`, you can't go back!**

If you aren't satisfied with the build tool and configuration choices, you can `eject` at any time. This command will remove the single build dependency from your project.

Instead, it will copy all the configuration files and the transitive dependencies (webpack, Babel, ESLint, etc) right into your project so you have full control over them. All of the commands except `eject` will still work, but they will point to the copied scripts so you can tweak them. At this point you're on your own.

You don't have to ever use `eject`. The curated feature set is suitable for small and middle deployments, and you shouldn't feel obligated to use this feature. However we understand that this tool wouldn't be useful if you couldn't customize it when you are ready for it.

## Learn More

You can learn more in the [Create React App documentation](https://facebook.github.io/create-react-app/docs/getting-started).

To learn React, check out the [React documentation](https://reactjs.org/).

### Code Splitting

This section has moved here: [https://facebook.github.io/create-react-app/docs/code-splitting](https://facebook.github.io/create-react-app/docs/code-splitting)

### Analyzing the Bundle Size

This section has moved here: [https://facebook.github.io/create-react-app/docs/analyzing-the-bundle-size](https://facebook.github.io/create-react-app/docs/analyzing-the-bundle-size)

### Making a Progressive Web App

This section has moved here: [https://facebook.github.io/create-react-app/docs/making-a-progressive-web-app](https://facebook.github.io/create-react-app/docs/making-a-progressive-web-app)

### Advanced Configuration

This section has moved here: [https://facebook.github.io/create-react-app/docs/advanced-configuration](https://facebook.github.io/create-react-app/docs/advanced-configuration)

### Deployment

This section has moved here: [https://facebook.github.io/create-react-app/docs/deployment](https://facebook.github.io/create-react-app/docs/deployment)

### `npm run build` fails to minify

This section has moved here: [https://facebook.github.io/create-react-app/docs/troubleshooting#npm-run-build-fails-to-minify](https://facebook.github.io/create-react-app/docs/troubleshooting#npm-run-build-fails-to-minify)
17 changes: 17 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,20 @@
# Installation

This project is designed to be simple to set up.

Requirements: `npm`

## Setup:

Clone the repository: `git clone https://github.com/Momentum-Team-14/social-e-cards-puss-in-boots-frontend.git`
Navigate to the repository root: `cd soc[TAB]`
Download all dependencies: `npm install`
Start the dev server: `npm start`

# Live Site

The site can be found live at `social-ecards-pib14.netlify.app`.

# Create a social greeting card service

**IMPORTANT**: Please read this document closely. Many resources are linked within it. Before starting on your project, assemble a list of the tasks you think you need to do and discuss how you will tackle these together. The front-end will need to make some basic [wireframes](https://www.orbitmedia.com/blog/7-reasons-to-wireframe/#:~:text=for%20your%20website.-,Wireframes%20are%20simple%20black%20and%20white%20layouts%20that%20outline%20the,focusing%20on%20a%20site's%20structure).
Expand Down
Loading