danspratling / portfolio-2019 Goto Github PK

This is something only required for my personal portfolio and might not be useful for others, and even if it is, the URL for it shouldn't be inlined.

Converting to a CMS based approach would be better.

There's some ambiguity in the naming convention of the titles of things. This indecisiveness has caused there to be conflicting naming conventions between 'heading' and 'title'.

All instances should be converted to 'title' to remove confusion.

Copy needs updating to better inform users of the purpose of the website and my talents. Current it is vague

https://danspratling.dev/contact

Was just checking out your portfolio website and noticed you had a typo.

Hello would you please share .env example as it is failing due to env variables.
thanks.

With issues with the font via gatsby cloud and issues automating the creation of cards, it makes sense to change approach.

Using Cloudinary and Netlify functions to handle the image generation will likely be cleaner than doing it in code and hopefully resolve both of the issues above.

Currently articles are retrieved from the footer in a static manner.
The rest of the footer is also created statically, which isn't a great experience for editing.

Converting this to a more flexible CMS driven means would be better.

Contentful was used originally as a way to manage content easily.

However, this website also wasn't built with the intention of making it open source at the time.

A markdown approach, where content can be stored within the repo makes far more sense, as does using a CMS built to manage markdown such as forestry.io.

This change cannot be completed until the launch of my new portfolio website

Right now calendly and analytics are within the code of the project.

The approach should be changed so that tags are separated from the project, either via GTM or an env file (or both).

The design for the contact form looks out of place, and should be updated to better match the design of the site.

This is especially important for the mailing list which currently cannot be used in many places due to its design.

Links using 'mailto:' do not link correctly, and are instead prefixed using the current page url. These urls should be absolute, not relative.

Testing is missing from the site, which should exist.

At a minimum

• forms should be tested for submission
• Buttons should be tested to ensure they're clickable
• Links should be tested to ensure they're clickable

Should also add in actions so tests are run on deploy.

Currently, this is very manual, having to call the function for each page individually with custom text.

Each page should have the function run automatically, and use the SEO text for the image title.