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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃帀 First implementation of the website with Ember.js #175

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

frinyvonnick
Copy link
Owner

@frinyvonnick frinyvonnick commented Jan 21, 2020

I implemented a naive version of the gitmoji-changelog website.

gitmoji-changelog-website

To test it locally you must first start the server:

cd src/packages/gitmoji-changelog-server
export PORT=9999
yarn start

Then start the Ember.js app:

cd src/packages/gitmoji-changelog-website
yarn start

This is a first implementation. I need to do a lot of things before publish this website. A code review would be appreciated since I just started developing with Ember.js 馃檹

Copy link

@lisequesnel lisequesnel left a comment

Choose a reason for hiding this comment

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

馃帀 Congratulations ! It's a great start !
You even tried Ember Octane before me 馃槅!


{{outlet}}
</article>
</main>

Choose a reason for hiding this comment

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

there is a missing new line ;)

Comment on lines +24 to +26
.catch((err) => {
console.error(err)
})

Choose a reason for hiding this comment

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

no this.loading = false in catch part ?

@kefranabg
Copy link
Contributor

Bonjour,

J'adore ce que vous faites.

Bonne journ茅e.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
work in progress Still in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants