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

Write Guide for lockedDoor.md #257

Merged
merged 9 commits into from
Apr 27, 2024
Merged

Write Guide for lockedDoor.md #257

merged 9 commits into from
Apr 27, 2024

Conversation

vaasut
Copy link
Contributor

@vaasut vaasut commented Feb 27, 2024

Write step by step guide with images explaining how to create a locked door

Description

The locked door FAQ section was a stub. Wrote a guide with images explaining how to create a locked door.

Write step by step guide with images explaining how to create a locked door
The format for one of the images caused the build to crash. Attempt to fix it.
The crash seems to be related to the images placed side-by-side, but I'm not entirely sure. Hopefully this fixes it.
I combined the two separate images, so hopefully npm doesn't get upset and crash this time.
I'm not sure why the last image kept causing a crash. Removing it for now, and maybe someone can help me add it back later.
npm crashed even without the image, so that was not the problem. Trying to rewrite the bottom of the file in case there was a hidden character causing the crash.
Seeing if npm crashes without the last step written.
I'm not sure what's causing the npm crash. I'm just going to leave the file so it looks how I intended in the preview, and hopefully someone can help me fix the npm crash.
@vaasut
Copy link
Contributor Author

vaasut commented Feb 27, 2024

The documentation page looks good in the preview, but when I try to commit it, the npm build crashes. I'm not sure what is causing the crash, and don't want to keep making tiny changes just guessing at what could be the problem.

@vaasut
Copy link
Contributor Author

vaasut commented Feb 27, 2024

Came back after sleeping on it, and I was able to understand the problem. I just needed to close the img elements with "/>".

@le-doux
Copy link
Owner

le-doux commented Apr 27, 2024

this is a great addition to the docs - thank you @vaasut!

@le-doux le-doux changed the base branch from main to version-8-12 April 27, 2024 20:30
@le-doux le-doux merged commit fb4ec8d into le-doux:version-8-12 Apr 27, 2024
1 check passed
le-doux added a commit that referenced this pull request May 3, 2024
* Add regenerate button for blips (#236)

* Regenerate blips without creating a new one
* Regenerating a blip does not modify its name

* Write Guide for lockedDoor.md (#257)

* Write Guide for lockedDoor.md

Write step by step guide with images explaining how to create a locked door

* Fix build crash for lockedDoor.md

The format for one of the images caused the build to crash. Attempt to fix it.

* Fix another build crash in lockedDoor.md

The crash seems to be related to the images placed side-by-side, but I'm not entirely sure. Hopefully this fixes it.

* Add new image in lockedDoor.md

I combined the two separate images, so hopefully npm doesn't get upset and crash this time.

* Remove image in lockedDoor.md

I'm not sure why the last image kept causing a crash. Removing it for now, and maybe someone can help me add it back later.

* Add image back to lockedDoor.md

npm crashed even without the image, so that was not the problem. Trying to rewrite the bottom of the file in case there was a hidden character causing the crash.

* Removing Step 5 in lockedDoor.md

Seeing if npm crashes without the last step written.

* Update lockedDoor.md

I'm not sure what's causing the npm crash. I'm just going to leave the file so it looks how I intended in the preview, and hopefully someone can help me fix the npm crash.

* Add closing tags to img in lockedDoor.md

* v8.12 updates

---------

Co-authored-by: Alex Nash <[email protected]>
Co-authored-by: vaasut <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants