search

OperationPandoraTrigger / OPT-Client-Mod

2 stars 8 forks source link

Ace addons #21

Closed GNCLORD-MDB closed 5 years ago

GNCLORD-MDB commented 5 years ago

Für die OPT geänderte ACE Addons damit das ausladen und Abwerfen von Fahrzeugen klappt

senshi-x commented 5 years ago

Brauchst du wirklich alles hier? Reicht es nicht, nur in ace_cargo etwas anzupassen? Können wir die relevanten Werte und Skripte nicht vielleicht sogar mit einer einfachen CfgPatches überschreiben?

GNCLORD-MDB commented 5 years ago

Ich glaube nicht das man scripte, wie Fahrzeuge überschreiben kann. Zum anderen weiß ich nicht was Form da alles geändert hat

senshi-x commented 5 years ago

Wieso kommt der PR dann von dir und nicht von Form? :thinking:

Skript überschreiben müsste auch gehen, muss halt wirkliche xakt gleich heißen und gleichen prefix haben. Alternativ kann man einfach die Skriptaufrufe umlenken. Also dort, wo die jeweiligen Skripte aufgerufen werden, einfach auf unsere "Custom"-Version umlenken. Hab ich aber auch noch nicht gemacht.

GNCLORD-MDB commented 5 years ago

weil ich und From an den gleichen Addons gearbeitet haben. Das sind ausladen und Abwerfen beiden drin in den 2 Addons.

Und das ich auch nicht weiß ob das geht und wir absolut keine Zeit habe da auch noch rum zu probieren, sollte man das gesamte ACE Addon so auswechseln wie es ist. Damit klappt es 100%

senshi-x commented 5 years ago

Du erwartest doch nicht ernsthaft, dass ich mal eben 18.000 Zeilen Code review. Wenn jetzt aus Termindruck eine Pfuschänderung wie diese nötig ist und keine Zeit ist, es sauber zu implementieren, ist es das Mindeste, das ihr im Code markiert, wo ihr herumgebastelt habt.

Und ihr könnt ja auch problemlos gemeinsam an einem PR arbeiten...würde alles deutlich übersichtlicher machen: Erlaube form Collaborator-/Write-Zugriff auf deinen "ace_addons"-Branch. Dann kann er und du beide darauf pushen&pullen und damit diesen PR bedienen. Und für uns Reviewer ist klar, wer was wo gemacht hat.

GNCLORD-MDB commented 5 years ago

nö, ich erwate das du da nur auf den Grünen Knopf drückst :-) Collaborator-/Write-Zugriff wo mache ich das ?

senshi-x commented 5 years ago

nö, ich erwate das du da nur auf den Grünen Knopf drückst :-) Collaborator-/Write-Zugriff wo mache ich das ?

Wie in den Meetings besprochen, gemergt wird nur nach Prüfung. Wenn du mir etwas Nicht-Prüfbares vorlegst, kann ich den grünen Knopf nicht drücken :) .

Bzgl Collab-Zugriff:

https://help.github.com/en/articles/enabling-branch-restrictions

Form als Collaborator hinzufügen, dann mit Branch Restriction das Pushen nur auf den Branch erlauben. Sonst kann Form auf all deine Branches pushen.

GNCLORD-MDB commented 5 years ago

sorry ,aber ich verstehe nicht was ich da machen soll, ich alles nur auf englisch und unverständlich für mich.

formtapez commented 5 years ago

Das mit ACE gehört hier nicht rein!

Frozen-byte commented 5 years ago

Danns schließe ich hier mal @formtapez wäre super wenn du bei ACE den PR aufmachst, wenn du Hilfe dafür benötigst wende dich an mich oder senshi.