Telegram macOS istemcisinin yeni açık kaynak deposunun bağlantısı beta test kullanıcılarıyla paylaşıldığı gün, istemci ekibi üçüncü taraf geliştiricilerden gelen düzeltmeleri kabul etmeye başladı. 25 Ekim 10:19 UTC itibarıyla, TelegramMessenger/telegram-macos adresine 19 farklı yazardan 81 pull request gönderildi ve bunların 30'u ana dala birleştirildi. Kabul edilen düzeltmeler macOS beta sürümüne birkaç saat içinde ulaşıyor, hatta biri bir saatten kısa sürede ulaştı. Bu durum, Mac istemcisinin beta kanalında olanları ve kendi fork'larını yazmak yerine resmi istemciyi düzeltmek isteyen geliştiricileri ilgilendiriyor.
Kaç Düzeltme Gönderildi ve Kaçı Kabul Edildi
Pull request, bir deponun sahibinin kendi dalına birleştirebileceği veya reddedebileceği önerilen bir kod düzeltmesidir. İlk pull request 23 Ekim 14:18 UTC'de geldi, ilki ise 17:45 UTC'de birleştirildi: bu, ülkeler listesindeki koleksiyonluk numaraların yanında korsan bayrağı yerine elmas gösteren #3 idi. 23 Ekim'de 17 pull request, 24 Ekim'de 55 ve 25 Ekim'in ilk on saatinde ise 9 tane daha gönderildi.
25 Ekim 10:19 UTC itibarıyla:
| Durum | Pull Request Sayısı |
|---|---|
| Birleştirildi | 30 |
| Birleştirilmeden Kapatıldı | 14 |
| Açık | 37 |
Birleştirilen düzeltmeler 11 farklı yazar tarafından gönderildi ve toplamda 7 307 eklenen satır ve 1 089 silinen satır içeriyor. Tüm 30 düzeltme tek bir hesap, macosswift tarafından birleştirildi. Profilinde Telegram macOS adı, Telegram şirketi ve macos.telegram.org web sitesi belirtilmiştir. Bir yazarın birleştirilen dokuz düzeltmesi codex/ önekiyle başlayan dallardan geldi; OpenAI'nin Codex ajanı bunları varsayılan olarak bu şekilde adlandırıyor.
İkinci Günde Ortaya Çıkan Kurallar
24 Ekim 16:12 UTC'de depoda CONTRIBUTING.md ve bir pull request şablonu ortaya çıktı. Her pull request'te artık dört bölüm bulunmalıdır: ana değişiklik, nedeni, aynı senaryoda "önce" ve "sonra" ekran görüntüleri veya videosu ve doğrulama açıklaması. Görünür bir sonuç yoksa, ekran görüntüleri yerine neden açıklanmalı ve doğrulama çıktısı eklenmeli; davranış değişiklikleri için ise macOS sürümü ve istemci derleme numarası belirtilmelidir. Herhangi bir bölümü eksik olan veya doğrulama onayı işaretlenmemiş bir pull request değerlendirmeye hazır değildir.
Şablon aynı akşam uygulanmaya başlandı. 17:58'den 18:09 UTC'ye kadar dört pull request REJECTED etiketiyle kapatıldı: "önce" ve "sonra" altında ne ekran görüntüsü ne de video yoktu. 19:31'den 19:36 UTC'ye kadar aynı açıklama 11 pull request'e daha bırakıldı. Bunlar kapatılmadı, tamamlanmaları istendi ve ikisi aynı akşam birleştirildi.
Neden Reddediliyorlar
Boş şablon dışında, düzeltmeler esas olarak, yorumda bir açıklama ile kapatılıyor:
- #6, düz yan panel ve favori klasör sekmeleri, arayüzü ve ürünü değiştiriyor ve ekip "şu anda bu tür değişiklikleri almayı planlamıyor";
- #13 AI düzenleyici düğmesini 10. karakterden itibaren gösteriyordu, ancak tasarıma göre üç satırdan sonra görünmesi gerekiyordu. Giriş alanı bu sırada 30 nokta yukarı zıplıyordu. Gönderilen mesajları düzenleme kısmı işlevsel kabul edildi ve ayrı olarak #22 numaralı pull request'e birleştirildi;
- #53 yalnızca Rusça dizeleri manuel olarak düzeltiyordu, ancak istemci çevirileri translations.telegram.org adresinde tüm diller için aynı anda yapılıyor;
- #23, yerel derlemeler ve uygulama imzası: derleme ve imza ayarlarını ekibin kendisinin değiştireceğine söz verildi ve bu düzeltme CI'daki derleme testini bozuyordu;
- #32 klasörlerde kullanıcının üye olmadığı sohbetleri gösteriyordu ve #40 aramayı bozuyordu. Altında bozulmanın ekran görüntüsü ve videosu bırakıldı.
#14 altına ilk başta kodun derlenmediği yazıldı ve göndermeden önce derlemenin kontrol edilmesi istendi. Daha sonra birleştirildi. Yüzen bir tab bar içeren #75 de bir notla geri döndürüldü: birkaç geçişten sonra avatar seçim halkasından ayrılıyordu. Yazar bunu 07:39 UTC'de düzeltti ve 08:09 UTC'de düzeltme birleştirildi.
Düzeltmeler Beta Sürümüne Nasıl Ulaşıyor
macOS beta sürümü, "Bug fixes, minor improvements, and more" aynı ek açıklamasıyla güncelleme sürümleri aracılığıyla yayınlanır. Derlemeye üçüncü taraf düzeltmelerinin dahil olduğu, ikili dosyasından anlaşılabilir: kodlarındaki dizeler daha önce bulunmadıkları yerlerde görünür.
13.1.0.283377 derlemesi 24 Ekim 21:27 UTC'de yayınlandı. İlk kez #20 (sohbet dışa aktarma hataları artık sessizce kaybolmak yerine her sohbet için gösteriliyor) kaynaklı "Could not load message history" ve "Message history pagination did not advance" dizelerini, #10 (uygulama simgesi değişikliği) kaynaklı "Failed to persist the custom application icon at %@" dizelerini, #2 ve #27 (bildirim ayarları) ile #16 (bot komutları) kaynaklı hizmet adlarını içeriyor. Bu beş düzeltme 15:50'den 18:45 UTC'ye kadar birleştirildi, yani beta sürümüne 2 saat 43 dakika ile 5 saat 37 dakika arasında bir sürede ulaştılar.
13.1.0.283380 derlemesi 25 Ekim 09:06 UTC'de, #75'in birleştirilmesinden 57 dakika sonra yayınlandı. Bu düzeltmenin kodundan TabBarItemControl ve SelectionGesture adları bu derlemede ortaya çıktı.
Ekibin kendi kod kısmı beta sürümüne neredeyse yetişti. 23 Ekim 17:04 UTC'de "Merge branch 'release' into master (13.1)" commit'i ana dala eklendi, Info.plist'teki derleme numarası 283305'ten 283357'ye yükseldi, bir milyondan fazla satır eklendi. Beta o saate kadar zaten 283368 idi. Beta sürümünde HTML ve JavaScript ile mesajları çizen HTMLBubbles modülü açık kodda hala yok ve depodaki iki betik, oraya girmediğini kontrol ediyor.
Zaten Birleştirilenler
| Pull Request | Ne Değiştiriyor |
|---|---|
| #2 | Telegram iOS'taki gibi bildirim ayarları: kategoriler, istisnalar, tepkiler |
| #27 | Hikayeler hakkında bildirimler ve hikaye ayarları bölümü |
| #75 | iOS'taki gibi yüzen tab bar, gösterge sürüklenebilir |
| #36 | Etkin olmayan oturumların otomatik sonlandırılması için "1 yıl" süresi |
| #31, #37 | "Klasörden Kaldır" öğesi ve klasör avatarının silinmesi |
| #22 | Gönderilen mesajı düzenlerken AI düzenleyici |
| #20 | Sohbet dışa aktarma hataları her sohbet için görünür |
| #34, #9 | Fotoğraf kopyalama, resmi panoya yerleştirir ve mesajlar tarihle birlikte kopyalanır |
| #29, #30 | Arayüzün bölgesel dile çevirisi (örneğin, pt-br) ve anketlerde, görevlerde ve kişi notlarında aksanlı harflerin girişi |
#75'te yazarın bir notu var: iOS'taki gibi göstergedeki lens efekti düzeltmeye dahil edilmedi. iOS'ta bu efekti, macOS SDK'sında bulunmayan kapalı bir UIKit bileşeni çiziyor ve bunu yalnızca Metal üzerinde kendi render'ınızla tekrarlayabilirsiniz.
