Plots2: Refactoring profile.html.erb Code pt.2

Erstellt am 19. Dez. 2018  ·  4Kommentare  ·  Quelle: publiclab/plots2

Erstes Mal?

Dies ist ein Problem nur für Erstbesucher . Das bedeutet, dass wir daran gearbeitet haben, es für Leute lesbarer zu machen, die entweder noch nie zu unserer Codebasis beigetragen haben, oder sogar für Leute, die noch nie zu Open Source beigetragen haben .

Wenn das Sie sind, sind wir daran interessiert, Ihnen beim ersten Schritt zu helfen, können Fragen beantworten und Ihnen dabei helfen. Beachten Sie, dass wir besonders an Beiträgen von Personen aus Gruppen interessiert sind, die in freier und Open-Source-Software unterrepräsentiert sind!

Wir wissen, dass der Prozess der Erstellung eines Pull-Requests die größte Hürde für neue Mitwirkende ist. Diese Ausgabe ist für dich

Wenn Sie schon einmal einen Beitrag geleistet haben, ziehen Sie in Betracht, diesen Beitrag jemandem Neuem zu überlassen und unsere allgemeinen Hilfesuchen- Probleme durchzusehen. Vielen Dank!

🤔 Was Sie wissen müssen.

Nichts. Diese Ausgabe soll Sie bei Open Source willkommen heißen :) Wir freuen uns, Sie durch den Prozess zu führen.

Das Problem

Unser Code in app/views/users/profile.html.erb ist sehr ausführlich... wir könnten etwas Refactoring gebrauchen

Wir können damit beginnen, eine sich wiederholende Zeile aus dem Code umzugestalten. Vorerst die erste Zeile von:

https://github.com/publiclab/plots2/blob/1931439c0c799b3e3420a9dcaace7a54cf895bd4/app/views/users/profile.html.erb#L216 -L223

Lösung

Was soll geändert werden:

Refaktorieren Sie die Zeile im Codeblock im Abschnitt "Das Problem", um stattdessen Ihre neue Hilfsmethode zu verwenden.

Diese Zeile kann auf <% if current_profile_user %> aktualisiert werden

Vielen Dank!!

Schritte zur Behebung

  • [ ] Beanspruchen Sie dieses Problem mit einem Kommentar hier unten und stellen Sie alle klärenden Fragen, die Sie benötigen
  • [ ] Forken Sie das Repository ab und richten Sie es lokal ein, indem Sie die README-Anweisungen für das Hauptrepo befolgen https://github.com/publiclab/plots2

    • [ ] Erstellen Sie einen neuen Feature-Branch mit einem eindeutigen Namen, der das Problem beschreibt

  • [ ] Versuchen Sie, das Problem mit den obigen Schritten zu beheben, aber noch bevor Sie fertig sind, können Sie:
    Übertragen Sie Ihre Änderungen in Ihren Zweig und starten Sie einen Pull-Request (siehe Beitrag zur Public Lab-Software ), aber markieren Sie ihn als "in Bearbeitung", wenn Sie Fragen haben oder noch nicht fertig sind
  • [ ] Verweisen Sie auf dieses Problem in Ihrem Pull-Request-Text
  • [ ] Sobald Sie Ihre Pull-Anfrage eingereicht haben, wird eine zusätzliche Checkliste für die Zusammenführung bereitgestellt

💬 Holen Sie sich Hilfe

Wenn Sie Hilfe benötigen - hier sind einige Optionen:

first-timers-only help wanted

Hilfreichster Kommentar

Ich muss noch einen Open-Source-Beitrag leisten, also würde mich das aufwärmen. Darf ich das behaupten?

Alle 4 Kommentare

Ich muss noch einen Open-Source-Beitrag leisten, also würde mich das aufwärmen. Darf ich das behaupten?

Hallo @dave-light, bitte mach weiter. Und willkommen bei Public Lab und Open Source World :globe_with_meridians: :earth_americas: .

@dave-light: Bist du dabei? oder darf das behaupten?

Hey @mzmmlds , ich bin dabei.

War diese Seite hilfreich?
0 / 5 - 0 Bewertungen