Skip to content

Commit

Permalink
docs: update README.md
Browse files Browse the repository at this point in the history
  • Loading branch information
michaelbrusegard authored Oct 7, 2023
1 parent a480c9c commit 234e4a8
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,7 @@ I would not recommend for anybody to use this code for their own website. It is
2. **Databases are great and I should have used one.**
- I have learned that databases are great and that I should have used one for this project. It would simplify a lot of the logic to have a database entry for each shop item that references the price, the compressed photo and the original photo. It would also make it easier to add more information to each shop item. For instance adding keywords for a search function.
3. **I should have used less Storage Buckets.**
- Using a seperate storage bucket for eveyr page is very unneccesary. It creates a bunch of environment variables that need to be set and it makes it harder to add new pages. It would have been jsut as simple to use a single STorage bucket for the showcase pages that has a directorys tructure referencing the different pages.
- Using a seperate storage bucket for every page is very unneccesary. It creates a bunch of environment variables that needs to be set and it makes it harder to add new pages. It would have been just as simple to use a single storage bucket for the showcase pages that has a directory structure referencing the different pages.
4. **Security is very interesting.**
- Setting up a secure website with user authentication and authorization for the admin panel has been very fun. I have learned a lot about how to secure a website and how to handle user authentication and authorization. I used sessions and cookies to authenticate Niclas and I used a middleware to authorize. I had to use CSRF tokens to prevent CSRF attacks. For the next project I will look into having multiple users and token based authentication.
5. **Using Google Cloud Platform was overkill.**
Expand Down

0 comments on commit 234e4a8

Please sign in to comment.