OperationPandoraTrigger / OPT-Client-Mod

2 stars 8 forks source link

Version 2.83 #19

Closed GNCLORD-MDB closed 5 years ago

GNCLORD-MDB commented 5 years ago
Frozen-byte commented 5 years ago

@WGPSenshi übernimmst du das Mergen? Würde das gerne spätestens Freitag ins Repo übernehmen.

GNCLORD-MDB commented 5 years ago

Ich frage wie das seine kann wenn ich nur Modupdate geschrieben haben. Da ist doch irgendwas bei übernehmen das letzten PR bei euch schief gelaufen.

senshi-x commented 5 years ago

Da ist nix schief gelaufen. Kannst die mod.cpp ja selbst vergleichen. Aktuell ist im Master eine Version mit "2.82" . Dein PR hat jetzt "2.83", aber der vorige commit in deinem Branch hat noch "2.81" gehabt. Damit weiß Github, dass irgendwelche Commits dazwischen fehlen, und sperrt deshalb vorsichtshalber einen Merge.

Einmal Pull from upstream/master und in deinen master-branch mergen, dann ist alles wieder gut. Solltest nach dem Pull auch sehen, was sich an Änderungen tut.

GNCLORD-MDB commented 5 years ago

Das ist irgendwas beim letzten PR schief gelaufen, ich kann keine andere Mod.cpp hochladen als die auf den Server ist. Da kommt immer eine Konflikt Meldung dann. Geht dann erster nach den diese PR eingefügt wurde und man eine kleine neuen aufmacht.

GNCLORD-MDB commented 5 years ago

kann doch garn nicht seinen , V2.82 war komplett fertig. Das ist der PR nicht richtig eingefügt worden deshalb stehen hier auch so viele alte Dinge noch drin ! Ich fummel da mal herum das das vielleicht geht

GNCLORD-MDB commented 5 years ago

Ich kann keine Pull vom Orginal machen ,da haben ich nix zum auswählen bei mir im VSC

senshi-x commented 5 years ago

Mrap baseclass beinhaltet Qilin/Prowler nicht, glaube ich. Habe gerade nur Handy, da ist nachgucken doof. Ob es alle haben sollen, ist nicht unsere Entscheidung. Konzept Team hat Ifrit, Hunter, Prowler, Qilin freigegeben, nicht mehr. Also entweder mit denen debattieren oder ändern ;) .

GNCLORD-MDB notifications@github.com schrieb am Do., 8. Aug. 2019, 16:34:

@GNCLORD-MDB commented on this pull request.

In addons/opt_vehicles/cars/config.cpp https://github.com/OperationPandoraTrigger/OPT-Client-Mod/pull/19#discussion_r312070034 :

class Car_F : Car

{

  • TFAR_hasIntercom = 1;

Wenn wirklich nur die Fahrzeuge haben sollen die und keine anderen haben dürfen ,dann kann ich da besser in die MRAP Basisclasse das kein machen. Aber warum ist das ein Problem wenn, das alle Rad Fahrzeuge hätten. Das macht doch überhaupt nix aus.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/OperationPandoraTrigger/OPT-Client-Mod/pull/19?email_source=notifications&email_token=ABTHNRYSVIUHMGRW6I665FDQDQVG7A5CNFSM4IJK4TZKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCA74VSA#discussion_r312070034, or mute the thread https://github.com/notifications/unsubscribe-auth/ABTHNR43WHL5UFGJVXF47RTQDQVG7ANCNFSM4IJK4TZA .

GNCLORD-MDB commented 5 years ago

Du schneidest auch die Hecke mit dem Lineal :-)

senshi-x commented 5 years ago

Vielen Dank :D Ich Schaue es mir morgen früh an, dann sollte alles passen :)

GNCLORD-MDB notifications@github.com schrieb am Do., 8. Aug. 2019, 17:59:

@GNCLORD-MDB commented on this pull request.

In addons/opt_vehicles/cars/config.cpp https://github.com/OperationPandoraTrigger/OPT-Client-Mod/pull/19#discussion_r312115859 :

class Car_F : Car

{

  • TFAR_hasIntercom = 1;

InterCom auf Hunter/Irif/Strider beschränkt, damit du ruhig schlafen kannst Senshi

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/OperationPandoraTrigger/OPT-Client-Mod/pull/19?email_source=notifications&email_token=ABTHNR37VQJ3MIGRWEHWXWDQDQ7GBA5CNFSM4IJK4TZKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCBALFEY#discussion_r312115859, or mute the thread https://github.com/notifications/unsubscribe-auth/ABTHNR5AP5STKA4Z7RKB6SLQDQ7GBANCNFSM4IJK4TZA .

GNCLORD-MDB commented 5 years ago

Ich versuche es, bin aber auf der Arbeit