- 
                Notifications
    
You must be signed in to change notification settings  - Fork 7
 
Resolução do exercicio #34 #35
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
base: desafios
Are you sure you want to change the base?
Conversation
| 
           Muitos commits juntos rsrs.. Deixa testar aqui o código e avaliar.  | 
    
| 
           Tenho que criar uma nova branch na proxima vez né  | 
    
| <td>".$base['app'].'</td> | ||
| <td>'.$base['email'].'</td> | ||
| <td><a href='.$base['url'].'>Open Site</a></td> | ||
| <td><a target="_blank" href='.$base['url'].'>Open Site</a></td> | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
target="_blank" é meio inseguro de usar 💥.. Viu a minha sugestão na outra vez @acidiney?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Claro que vi, isso é do exercicio antigo nem retifiquei mais kkkk
        
          
                app/UpdateFileJson/index.html
              
                Outdated
          
        
      | <hr /> | ||
| <p class="red"> * PREVIEW</p> | ||
| </div> | ||
| <iframe src="http://dojo-php.herokuapp.com/ConvertjsonToArray/" width="100%" height="600" style="border:0px"></iframe> | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Há mesmo necessidade deste iframe @acidiney ??
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Era para retornar já a lista do usuario, só para confirmação... Se achares que não posso remover
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No teste que fiz, os o iframe não era atualizado. Se calhar fica melhor não chamando o iframe, apena informar que o usuário foi adicionado e pedir para pessoa voltar à lista/página onde pega todos membros..
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
já resolvi isso faz tempo kkkkk
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tenta de novo ... ou use o dojo-php.herokuapp.com
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
        
          
                data/codingdojo.json
              
                Outdated
          
        
      | } | ||
| ] | ||
| } No newline at end of file | ||
| {"app":"Coding Dojo Angola","url":"http:\/\/codingdojoangola.com","email":"geral@codingdojoangola.com","membros":[{"nome":"Lucius Curado","cargo":"Moderador e Organizador","avatar":""},{"nome":"Jos\u00e9 Cage","cargo":"Moderador e instrutor","avatar":""},{"nome":"Doriel Fonseca","cargo":"Instrutor #dojojavascript","avatar":""},{"nome":"Pedro Massango","cargo":"Desenvolvedor Java","avatar":""},{"nome":"Eleandro Duzentos","cargo":"Desenvolvedor PHP e .Net","avatar":""},{"nome":"Renato Martins","cargo":"Desenvolvedor Java e PHP","avatar":""},{"nome":"Victor Kiffen","cargo":"Desenvolvedor Java","avatar":""},{"nome":"Jetro Domingos","cargo":"Desenvolvedor Java","avatar":""},{"nome":"Acidiney Dias","cargo":"Desenvolvedor PHP","avatar":""},{"nome":"Nelson J\u00fanior","cargo":"Desenvolvedor PHP","avatar":""},{"nome":"Masiry Link","cargo":"Desenvolvedor PHP","avatar":""},{"nome":"Aleixo Jeronimo","cargo":"","avatar":""},{"nome":"Cuz\u00e3o","cargo":"aaa","avatar":""},{"nome":"aa","cargo":"aaa","avatar":""}]} No newline at end of file | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Porquê uma palavra obscena na lista de membros? 👎
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No caso o último item dentro da chave/array membros..
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Era só os meus testes kkkkkkkk, preguiça aqui
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh kkkk tavas a falar do cuzão kkkkkkkkkk, kkkkkkkkk eu tava vendo um video br quando tava fazendo ai escreve sem percebr kkkkkkk
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tem que prestar atenção antes do commit meu jovem. Pois o termo sendo ofensivo faz-nos passar vergonha "ao vivo" no evento @acidiney
😸
| // In the end, I'm add the new data in my file original | ||
| if(file_put_contents($this->source, $this->alterFile)): | ||
| return json_encode(["message" => "File updated with success!"], 201); else: | ||
| return json_encode(["message" => "File not was updated!"], 400); | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
O correto neste caso é "File was not updated!"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Concordo, muita inteligência aqui
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Só prestar atenção rsrs..
This commit consists of patches automatically generated for this project on https://scrutinizer-ci.com
Scrutinizer Auto-Fixes
          
 Foi o que sugeri tempos atrás lá no CDA.. Um branch pra cada desafio e depois dar merge em   | 
    
No description provided.