Skip to content

Quick fix from Jade to Pug - #69

Open
Extarys wants to merge 4 commits into
neumino:masterfrom
Extarys:master
Open

Quick fix from Jade to Pug#69
Extarys wants to merge 4 commits into
neumino:masterfrom
Extarys:master

Conversation

@Extarys

@Extarys Extarys commented Jul 5, 2017

Copy link
Copy Markdown

Tested and working.

The only thing that didn't work is the "Add field" in a table for an unknown reason.
Field doesn't get created or Express put a console error with an undefined value.

Everything else works.

@Extarys

Extarys commented Jul 5, 2017

Copy link
Copy Markdown
Author

Added .btn-primary or .btn-delete classes on primary action buttons for better UX.

@sukrosono

Copy link
Copy Markdown

Awesome, but why it's add rethinkdb as dependency?

@Extarys

Extarys commented Sep 27, 2017

Copy link
Copy Markdown
Author

It jumps from "rethinkdb": ">= 1.12.0-0" to "rethinkdb": "^2.3.3". it's a huge jump but it was already a dependency though since it needs a driver to connect and manage your database stuff. It's like, you need mysql driver for phpmyadmin. :P

See file package.json for dependency changes.

Note that it was very quickly tested and I cannot guarantee anything. Feel free to download and play around. Also, I'm not a pull/push professional, some mistake may have been make. (Never really used git before 2017 and I mostly comment)

Note that I did a couple of small typo changes a second ago directly in this. (Bump to 0.4 too since we change engine)

@sukrosono

Copy link
Copy Markdown

@Extarys ok i got it if it was previously there, and with your explanation. You are awesome, i also still new to git. I am still unable to install chateau on my machine 😞 . It throw Node.js EACCES after i change the chateau config.

@Extarys

Extarys commented Sep 27, 2017

Copy link
Copy Markdown
Author

THANKS :D I needed this - too much saltiness yesterday on github. I had similar issue (under linux) npm had access issue maybe npm permissions fix will help (make sure this is your issue first) . If you every need help or just chat, ping me, I need friends :P

@sukrosono

Copy link
Copy Markdown

yay i need friends too, maybe we can chat at gitter sometime. I got error EACCES on using port, i guest the express port. i am gona leave it for now.

@Extarys

Extarys commented Sep 28, 2017

Copy link
Copy Markdown
Author

Oh you can just search i the files and change it for like 3010. It might be another nodejs app.

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.

2 participants