New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Irrelevant link in the error message #735

Closed
zwhitchcox opened this Issue Oct 22, 2016 · 4 comments

Comments

Projects
None yet
3 participants
@zwhitchcox

zwhitchcox commented Oct 22, 2016

so, I still haven't debugged my error, but the error was that somehow I gave the wrong type to my port, and I get this link

http://guide.elm-lang.org/effect_managers/

asking me to please follow the guidelines there, which have absolutely nothing to do with the fact that I gave the port the wrong type. Please put some relevant information in there or change the link, because I just wasted about five minutes reading that, when I could have been working on solving my error.

@process-bot

This comment has been minimized.

Show comment
Hide comment
@process-bot

process-bot Oct 22, 2016

Thanks for the issue! Make sure it satisfies this checklist. My human colleagues will appreciate it!

Here is what to expect next, and if anyone wants to comment, keep these things in mind.

process-bot commented Oct 22, 2016

Thanks for the issue! Make sure it satisfies this checklist. My human colleagues will appreciate it!

Here is what to expect next, and if anyone wants to comment, keep these things in mind.

@zwhitchcox

This comment has been minimized.

Show comment
Hide comment
@zwhitchcox

zwhitchcox Oct 22, 2016

   23 port setStorage : Model -> Html msg                                                                                                  
   24                                                                                                                                      
   25                                                                                                                                      
   26 updateWithStorage : Msg -> Model -> Cmd -> ( Model, Cmd )                                                                            
   27 updateWithStorage msg model cmds =                                                                                                   
   28   let                                                                                                                                
   29     newModel =                                                                                                                       
   30       update msg model                                                                                                               
   31   in                                                                                                                                 
   32     ( model, Cmd.batch [ setStorage newModel, cmds ] ) 

This is the code, by the way, working on reproducing the todo list from memory. Not looking for anyone to solve my problem, just in case the code is relevant.

zwhitchcox commented Oct 22, 2016

   23 port setStorage : Model -> Html msg                                                                                                  
   24                                                                                                                                      
   25                                                                                                                                      
   26 updateWithStorage : Msg -> Model -> Cmd -> ( Model, Cmd )                                                                            
   27 updateWithStorage msg model cmds =                                                                                                   
   28   let                                                                                                                                
   29     newModel =                                                                                                                       
   30       update msg model                                                                                                               
   31   in                                                                                                                                 
   32     ( model, Cmd.batch [ setStorage newModel, cmds ] ) 

This is the code, by the way, working on reproducing the todo list from memory. Not looking for anyone to solve my problem, just in case the code is relevant.

@zwhitchcox

This comment has been minimized.

Show comment
Hide comment
@zwhitchcox

zwhitchcox Oct 22, 2016

This is the finished code

   25 port setStorage : Model -> Cmd msg                                                                                                   
   26                                                                                                                                      
   27                                                                                                                                      
   28 updateWithStorage : Msg -> Model -> ( Model, Cmd )                                                                                   
   29 updateWithStorage msg model =                                                                                                        
   30   let                                                                                                                                
   31     ( newModel, cmds ) =                                                                                                             
   32       update msg model                                                                                                               
   33   in                                                                                                                                 
   34     ( newModel, Cmd.batch [ setStorage newModel, cmds ] ) 

with that particular error solved...I found the answer to my problem (or at least the one that solved the error) here

Please consider changing this link to this url.

zwhitchcox commented Oct 22, 2016

This is the finished code

   25 port setStorage : Model -> Cmd msg                                                                                                   
   26                                                                                                                                      
   27                                                                                                                                      
   28 updateWithStorage : Msg -> Model -> ( Model, Cmd )                                                                                   
   29 updateWithStorage msg model =                                                                                                        
   30   let                                                                                                                                
   31     ( newModel, cmds ) =                                                                                                             
   32       update msg model                                                                                                               
   33   in                                                                                                                                 
   34     ( newModel, Cmd.batch [ setStorage newModel, cmds ] ) 

with that particular error solved...I found the answer to my problem (or at least the one that solved the error) here

Please consider changing this link to this url.

@jvoigtlaender

This comment has been minimized.

Show comment
Hide comment
@jvoigtlaender

jvoigtlaender Oct 22, 2016

Contributor

This is already fixed for the next compiler version.

Contributor

jvoigtlaender commented Oct 22, 2016

This is already fixed for the next compiler version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment