search

theman550 / DAT257-K9

Project for course DAT257
1 stars 1 forks source link

Add trip with all fixes #15

Closed ITJohan closed 4 years ago

ITJohan commented 4 years ago

Här är Mohammeds branch fast med alla fixar.

ITJohan commented 4 years ago

Har för mig testing moduler ska sparas under "dev-dependencies", att man installerar under --save-dev. Har ingen verklig påverkan ändå tror jag.

Håller med, ändrar det. Edit: såg att de låg med i både dev och vanliga.

Indentationen ser lite underlig ut, men det är nog för att vi den nya eslintrc.json. Rekommenderar att vi drar in min ESLint PR

Jag skrev en kommentar i din PR :relaxed:

ITJohan commented 4 years ago

Nu är det fixat så du får gärna acceptera denna PR om det inte var något annat @AndreasWJ

BasharDimaria commented 4 years ago

don't merge it , det finns ett fel verkligen i Design , när jag vill göra post task till Serven , jakob skickade till mig datamodell som inte stämmer alls vad Gui har för component , det finns igen fält till price eller decription och det finns bara starttime och det vet inte exakt vad är det för nogåt , så jag har ändrat design helt och hållet och det för att passa datamodell men jag skickat många frågor till jakob och jag väntar på svaret för att vara säker att det inte missas nogåt , jag ska pusha ändringar så fort jag får svaret.

ITJohan commented 4 years ago

don't merge it , det finns ett fel verkligen i Design , när jag vill göra post task till Serven , jakob skickade till mig datamodell som inte stämmer alls vad Gui har för component , det finns igen fält till price eller decription och det finns bara starttime och det vet inte exakt vad är det för nogåt , så jag har ändrat design helt och hållet och det för att passa datamodell men jag skickat många frågor till jakob och jag väntar på svaret för att vara säker att det inte missas nogåt , jag ska pusha ändringar så fort jag får svaret.

Yes, det är bara att pusha till add-trip branchen så uppdateras denna PR.

BasharDimaria commented 4 years ago

Jag kunde inte pusha bara ändringar till add-trip with all fixes , jag vet inte varför , så i add trip gui ser precis som det sista commit som johan gjort på (add trip with all fixes ) med ändrinngar bara i Addtrip.js, så det finns två fil Addtrip och AddTrip , ändringar är i Addtrip , jag kunde inte ta bort den andra , så om ni kan göra det istället. jag ville merga först med addtrip with all fixes först och sen pusha ändrigar , jag har gjort så här ,

git merge origin add-trip

(fixat allt med min ändrigar ) git add . git commit -m 'new disgnä' git push origin add-trip

men det hände inte .

så läget är nu , ändringar ligger i request (Add trip Gui ) men inte mergat med add-trip för att jag är inte säkert att det inte ska riva add-trip helt och hållet :) så bara kollar på den om det är bra kan jag merga med add-trip anars skulle jag bara skicka AddTrip.js till Johan och han lägger den i add-trip branch :)

ITJohan commented 4 years ago

Jag svarade i andra PR, se https://github.com/theman550/DAT257-K9/pull/29#issuecomment-697151783 :)

BasharDimaria commented 4 years ago

yes , bara Addtrip.js , men nu , hur kan jag clona igen från den senatse commit du ska göra i add-trip till min , för att push den andra task senare idag , (Post task) , jag ska göra en ny branch för den nya task , men jag vill hämta senaste ändring i add-trip , när jag mergat Addtrip_GUI med add-trip blev det kaos som du såg , hur kan jag fixa det . så hur kan ta add-trip with all fixes när du lägger Addtrip.js till min dator .

ITJohan commented 4 years ago

@BasharDimaria kan du acceptera denna nu och merga? Sen tar du bort branchen med hjälp av knappen som dyker upp.