Expose Msg with Msg(...) about toasty

[pablen]

Hello Gabriel! I'm a bit reluctant about exposing the whole Msg subtypes because then any internal change in the Msg implementation would imply a major version bump when publishing the package as explained here: https://medium.com/@ghivert/designing-api-in-elm-opaque-types-ce9d5f113033.

Maybe I can expose some helper functions like isTransitioning : Toasty.Msg -> Bool to let you extract the internal message.

Would that help with what you are trying to achieve?

[gabrielperales]

Hi @pablen!, I ended creating those helpers functions, and I think it is ok. I was trying to create my own toast component which was just exactly as yours but without having to pass the config record to each function. Here you can see the code and how I exposed those helper functions.

If you think there is an approach better to achieve the same thing, please let me know it :)

[pablen]

Oh, so I think I misunderstood what you were trying to do here.

In your case I think those helper functions are a bit redundant because you can use the Success, Warning and Error types as constructor functions without the need of defining new helper functions.

You can take a look at the demo code, you'll notice I'm using those types in such a way:

toasty/examples/src/App.elm

Lines 50 to 61 in e8c0a8f

	case Char.fromCode keycode of
	's' ->
	(model ! [])
	|> addToast (Toasty.Defaults.Success "Allright!" "Thing successfully updated")
	'w' ->
	(model ! [])
	|> addToast (Toasty.Defaults.Warning "Warning!" "Please check this and that.")
	'e' ->
	(model ! [])
	|> addToast (Toasty.Defaults.Error "Error" "Sorry, something went wrong...")

Also, look a few lines above how I defined a single addToast helper function to avoid having to pass the config record around every time:

toasty/examples/src/App.elm

Lines 37 to 39 in e8c0a8f

	addToast : Toasty.Defaults.Toast -> ( Model, Cmd Msg ) -> ( Model, Cmd Msg )
	addToast toast ( model, cmd ) =
	Toasty.addToast myConfig ToastyMsg toast ( model, cmd )

I think you'll be good to go with this!