-
Notifications
You must be signed in to change notification settings - Fork 79
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
Milestone 3A Aidan Nesbitt #157
Closed
Closed
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
…L elements to the blogTitle element
…, added navbar to layout.tsx - having issue with hydration for Resume page
…ges for each blog
…ly display associated information and link to correct pages
…n the footer and the bottom of the page
… up on the BlogPreview, with the blogs fetched from the database
…me and blog pages
…curring on portfolio page but have not solved yet
…rate title of the page (I assume I will update this more elegantly later)
MongoDB "entries" are called documents |
jlaksana
reviewed
Dec 6, 2023
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.
Looks good. Proceed to 3B!
jlaksana
reviewed
Dec 6, 2023
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.
Developer: Aidan Nesbitt
Closes #110
Pull Request Summary
Created a MongoDB Atlas database to store data that will be displayed on my website (blog Posts and portfolio content). Added a connectDB function that connects to the using my URL. Created getBlogs() and getPortfolios() functions that connect to the database with the connectDB function and retrieve blogs and portfolio (entries? objects? data points?) respectively. Call getBlogs() and getPortfolios() with await at top of each page's Home component, and then render their data within the returned HTML if the data request was successful. Implemented a check for each page (blogs and portfolios) that checks if the blogPosts/portfolios variable is not null and has a length greater than 0, and generates the content within the page if those conditions are met. If those conditions are not met, and the blogPosts/portfolios variable is null, then create a div with a nested paragraph on each page that says, "There was an error...". Also created blogSchema and portfolioSchema files for handling the data retrieved from the database.
Note:
There are hydration errors occurring on my portfolio page, and the error says it may have to do with head and HTML but I'm not entirely sure how to fix it – I couldn't find a solution on StackOverflow or ChatGPT just yet. I found that hydration errors can occur if you improperly situate HTML tags (such as with nested a paragraph within another paragraph), and I think that this error might be occurring with the way that I am generating the dynamic content (but don't know how to fix).
Special Instructions (optional)
Pull Request Checklist
Screenshots/Screencast