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

Finish initial Netlify CMS setup #10

Merged
merged 1 commit into from Dec 15, 2022
Merged

Finish initial Netlify CMS setup #10

merged 1 commit into from Dec 15, 2022

Conversation

yndajas
Copy link
Collaborator

@yndajas yndajas commented Dec 10, 2022

Note/question: I noticed that there was already a UX design (/ux) folder with a couple of draft pages inside. Is this intended to be the interaction design folder? If so, shall I remove the /interaction-design folder I added and just add the /ux one to the Netlify CMS config instead?


This adds folders and initial index files for top-level content, development and interaction design sections

It also updates the admin config to include these three sections, and to use file/folder paths with src/ included. The latter is from a change to the Playbook's admin config in commit 9eb5e7f, which fixed an issue where content was not displayed or editable when opening a page for editing in the CMS (though this was not reproducible locally)

@see dxw/playbook#1001

How to test in the browser

Per dxw/playbook#1001, the easiest way to test that this works is to go directly to the CMS/admin area of the deploy preview, e.g.: https://deploy-preview-10--startling-pasca-a7e8b4.netlify.app/admin/#/collections/home

You could also run it locally using the instructions in src/admin/config.yml

@netlify
Copy link

netlify bot commented Dec 10, 2022

Deploy Preview for startling-pasca-a7e8b4 ready!

Name Link
🔨 Latest commit 44cf453
🔍 Latest deploy log https://app.netlify.com/sites/startling-pasca-a7e8b4/deploys/639b0db85f86310008c3b9d4
😎 Deploy Preview https://deploy-preview-10--startling-pasca-a7e8b4.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@dxw dxw deleted a comment from netlify bot Dec 10, 2022
Copy link
Collaborator

@calumryan calumryan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍🏻 Yes, correct that interaction design should be the folder name

This adds folders and initial index files for top-level content, development and
interaction design sections. The contents of the previous UX folder have been
moved into the interaction design folder

It also updates the admin config to include these three sections, and to use
file/folder paths with `src/` included. The latter is from a change to the
Playbook's admin config in commit 9eb5e7f, which fixed an issue where content was
not displayed or editable when opening a page for editing in the CMS (though this
was not reproducible locally)

@see dxw/playbook#1001
@yndajas yndajas merged commit 7df67e9 into main Dec 15, 2022
@yndajas yndajas deleted the initial-netlify-setup branch March 16, 2023 17:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants