search

theman550 / DAT257-K9

Project for course DAT257
1 stars 1 forks source link

Add trip gui #29

Closed BasharDimaria closed 4 years ago

BasharDimaria commented 4 years ago

Jag kunde inte pusha bara ändringar till add-trip with all fixes , jag vet inte varför , så i denna branch 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 .

ITJohan commented 4 years ago

Ja det ser ut som att något galet har skett. Jag tror att problemet är att du utgått från den ouppdaterade ADDTrip_GUI branchen och vill merga med add-trip branchen som baseras på förstnämnda. Det ser inte ut som du fått med mina fixar så jag tror lättaste är om jag fixar till add-trip branchen med de nya ändringarna. Är det bara Addtrip.js som du har ändrat eller?

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 .

ITJohan 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 .

Det blir att den andra pull requesten mergas med master, ska fixa det snart. Sen skriver du

git checkout master
git pull
git checkout -b add-trip-post (eller vad du nu vill kalla branchen)

Då har du en ny branch från den senaste mastern.

BasharDimaria commented 4 years ago

har ni mergat ändrinar till Master ? om jag gör pull från master kommer jag få Hallo World proj annars hhhhhhhh .

BasharDimaria commented 4 years ago

jag har kollat nu på mastern , det finns inte Addtrip.js eller test i denna.

ITJohan commented 4 years ago

Jag har fixat till den andra PRn nu så du kan acceptera och merga, jag stänger denna nu.