axolotl-chat / axolotl

A Signal compatible cross plattform client written in Go, Rust and Vuejs
https://axolotl.chat
GNU General Public License v3.0
326 stars 52 forks source link

Make contributing easier #185

Open Flaburgan opened 4 years ago

Flaburgan commented 4 years ago

It should be as easy as possible to develop for Axolotl. And now is a good time to ask ourselves what could be improved as Hacktoberfest is about to start and that would hopefully bring new people.

Ideas (please complete):

olof-nord commented 4 years ago

Regarding the backend flags: The documentation is currently not up to date.

The available flags are as follows:

./axolotl --help
Usage of ./axolotl:
  -e string
        use either electron, ut, lorca, server or me
  -eDebug
        use to show development console in electron
  -host string
        Host to serve UI from. (default "127.0.0.1")
  -port string
        Port to serve UI from. (default "9080")
  -qml string
        The qml file to load. (default "qml/phoneui/main.qml")
olof-nord commented 4 years ago

It is not clear to me what QML file is expected by the -qml flag: What is the purpose and intention of this?

nanu-c commented 4 years ago

QT/QML provides a webengineview which is basically a chromium engine that is shipped with qt. With the -qml flag it's possible to use different qml files for the different integrations like Plasma Mobile, Lomiri or SailfishOs. -e ut is a shorthand for loading the ubuntu touch specific qml file

olof-nord commented 4 years ago

It should be as easy as possible to develop for Axolotl. And now is a good time to ask ourselves what could be improved as Hacktoberfest is about to start and that would hopefully bring new people.

Ideas (please complete):

* Improve the readme

  * Remove `Migrating from janimo/axolotl` part?
  * It's not clear what `Run flags` is about, if it's about the backend `go run .` command then it should be placed next to it
  * Remove `Installation on UT` as it is now available in the OpenStore, and that's indicated at the very beginning of the Readme?
  * Move the `Translations` paragraph as it is currently between `Installation of development environment` and `Run development`. Those two should probably be merged btw

* Fix `VUE_APP_WS_ADDRESS=10.0.0.2 npm run serve` as it looks like it can connect to the phone anymore. At least that's what Firefox console tells me

* Provide some fake data to load locally with all the different states of the application, as currently once launched the devs are stuck on the registering screen and can't continue if they don't have a phone number. Ideally, a secret route would load it, so we can try Axolotl on secondary phone for example
olof-nord commented 4 years ago

I think there is one more thing to do in regards to this.

In the main readme there are currently three sections, which all covers more or less the same topic: How to set up a local development environment.

I could see these three put together into one separate document at docs/CONTRIBUTING.md.