-
-
Notifications
You must be signed in to change notification settings - Fork 532
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
chore: split website data build process to several steps #830
Conversation
✅ Deploy Preview for asyncapi-website ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site settings. |
⚡️ Lighthouse report for the changes in this PR:
Lighthouse ran on https://deploy-preview-830--asyncapi-website.netlify.app/ |
ee82184
to
4843ff7
Compare
I don't remember why this PR at all anymore 😆 |
I think you wanted to refactor scripts, split the |
are we closing it? I prefer doing it the way we do it not, that each new script gets it's own folder, rather refactor all like this unless you have time to update the PR |
@derberg Yeah, I will close. We should try make this approach that you wrote, but in another PR. |
Description
That PR doesn't change anything in styling and similar things in our website. It only splits posts building process to several steps, each part like blog-posts, jobs etc now will be located in separate files, usually with another shapes to easier handle data in appropriate pages.
There is a lot of code, but it's because I had to change the way of getting some values - the keys in the objects have changed.