1082618 - [Messages] add a README file

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::SMS, defect)

Description

[Julien Wajsberg [:julienw]]

The README file should explain the basic steps to contribute to the Messages application.

Comment 1

[Julien Wajsberg [:julienw]]

Attachment: github PR

See https://github.com/julienw/gaia/blob/1082618-sms-readme/apps/sms/README.md for the formated file.

Comment 2

[Steve Chung [:steveck]]

Comment on attachment 8504794 [details] [review]
github PR

I think the information in this document is clear for debugging with Nightly. Only some few comments and nit on github, and some thoughts here:

- Should we also provide a link about how to debug with web IDE(app manager)?
- How to create a bug with clear and sufficient(log, image, video or even database) information? I think it's also a good way for contribution.

Comment 3

[Julien Wajsberg [:julienw]]

Comment on attachment 8504794 [details] [review]
github PR

Hey Steve,

I at last updated the README according to your comments!

I think we can make the part about Mulet better but we can maybe iterate from this in a separate bug?

I also added some troubleshooting information in [1], these are issues I saw on various computers during a hackathon, and I think it's better to have it on MDN than in the README :)

[1] https://developer.mozilla.org/en-US/Firefox_OS/Developing_Gaia/Running_the_Gaia_codebase

Comment 4

[Steve Chung [:steveck]]

Comment on attachment 8504794 [details] [review]
github PR

That a details and helpful intro for developer :) Only one nit on github(hope my grammar is correct).

Comment 5

[Julien Wajsberg [:julienw]]

Hey Chris, would you have the time to do a proof reading on https://github.com/julienw/gaia/blob/1082618-sms-readme/apps/sms/README.md ?

You can comment directly in https://github.com/mozilla-b2g/gaia/pull/25141/files

Also tell me if some parts could be more suitable for MDN than here? I tried to add some things in MDN as part of this bug already.

Thanks !

Comment 6

[Chris Mills [:cmills]]

I've included my comments at https://github.com/mozilla-b2g/gaia/pull/25141 as requested.

I've also done a full copy edit of the README; see attached to this bug in an .md file.

Comment 7

[Chris Mills [:cmills]]

Attachment: messages_readme_CE_chris_mills.md

Comment 8

[Julien Wajsberg [:julienw]]

landed in 17cd52956a545a8ef604a76f816b4ddb3e336802

Thanks all !