Skip to content

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> No newline at end of file

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.

4 participants