Open invitation setting

[floatdrop]

Implementation of #64 issue:

Todo:

• Fix html in templates
• Add migration script for open_invitation setting in database

[bradrydzewski]

a couple of things

on the invitation form, where you enter the email, you can use <input class="only-child" to get the styling right :)

second, I added a db migration package, but haven't hooked it up yet. It would be interesting if you could invoke from here:
https://github.com/drone/drone/blob/master/cmd/droned/drone.go#L57

third, any ideas on how to unit test migrations? :)

[floatdrop]

I will look into migrations today in different branch.

Thank you for the hints!

[bradrydzewski]

one minor comment here. The first time a user launches Drone, the Settings will not exist in the database, and will cause a panic:
https://github.com/floatdrop/drone/blob/6937b7b21e91b092ace26aeba8c17ca720f83160/pkg/handler/app.go#L50

maybe we should use

settings, _ := database.GetSettings()

and just make sure in the template we check for a nil settings?

[bradrydzewski]

maybe something like:

settings, err := database.GetSettings()
if err != nil {
    http.Redirect(w, r, "/install", http.StatusSeeOther)
}

what do you think?

[bradrydzewski]

I'll go ahead and fix the potential panic issue right now. Thanks for taking it this far :)

[floatdrop]

Thank you! I thought this commit will fix panic, but may be I miss something.

[bradrydzewski]

yes, you are right. your commit did fix it :)
thanks again!