Using new cwtch attributes and get/ret val requests #272

Слито
sarah слито 1 коммит(ов) из :keyval в master 2020-03-31 19:06:38 +00:00
dan прокомментировал(а) 2020-03-19 21:16:42 +00:00
Владелец
Описание отсутствует.
buildbot прокомментировал(а) 2020-03-28 00:26:21 +00:00
Участник

Drone Build Status: failure

https://build.openprivacy.ca/cwtch.im/ui/508

Drone Build Status: failure https://build.openprivacy.ca/cwtch.im/ui/508
buildbot прокомментировал(а) 2020-03-28 16:16:42 +00:00
Участник

Drone Build Status: failure

https://build.openprivacy.ca/cwtch.im/ui/509

Drone Build Status: failure https://build.openprivacy.ca/cwtch.im/ui/509
dan изменил(а) заголовок с WIP: Using new cwtch attributes and get/ret val requests на Using new cwtch attributes and get/ret val requests 2020-03-30 16:17:59 +00:00
buildbot прокомментировал(а) 2020-03-30 16:22:53 +00:00
Участник

Drone Build Status: success

https://build.openprivacy.ca/cwtch.im/ui/510

Drone Build Status: success https://build.openprivacy.ca/cwtch.im/ui/510
sarah запросил(а) изменения 2020-03-30 18:37:04 +00:00
go/handlers/peerHandler.go Устаревшее
@ -93,0 +99,4 @@
}
}
case event.NewRetValMessageFromPeer:
// TODO: update the ui
sarah прокомментировал(а) 2020-03-30 18:15:38 +00:00
Владелец

?

?
dan прокомментировал(а) 2020-03-30 20:57:27 +00:00
Автор
Владелец

oph, reminder to myself i didnt delete after I filled the case:

oph, reminder to myself i didnt delete after I filled the case:
go/ui/gcd.go Устаревшее
@ -331,0 +327,4 @@
if !exists {
zoom = "1.0"
}
locale, exists := the.Peer.GetAttribute(attr.GetSettingsScope(constants.LocaleSetting))
sarah прокомментировал(а) 2020-03-30 18:34:18 +00:00
Владелец

I feel we need a better way of managing defaults than this.

I feel we need a better way of managing defaults than this.
dan прокомментировал(а) 2020-03-30 21:00:24 +00:00
Автор
Владелец

we were doing nothing. open to suggestions. we don't really have a good handle yet on what our settings will ultimatly be, we need to work on our settings management some. Erinn has some ideas on this

we were doing nothing. open to suggestions. we don't really have a good handle yet on what our settings will ultimatly be, we need to work on our settings management some. Erinn has some ideas on this
go/ui/imageType.go Устаревшее
@ -0,0 +3,4 @@
import "encoding/json"
const (
TypeImageDistro = "distro"
sarah прокомментировал(а) 2020-03-30 18:34:53 +00:00
Владелец

wat? needs documentation

wat? needs documentation
go/ui/manager.go Устаревшее
@ -57,0 +69,4 @@
log.Infof("get nick for contact %v\n", id)
nick, exists := the.Peer.GetContactAttribute(id, attr.GetLocalScope(constants.Name))
if !exists {
log.Infof("local !exits\n")
sarah прокомментировал(а) 2020-03-30 18:35:26 +00:00
Владелец

remove infos

remove infos
go/ui/manager.go Устаревшее
@ -57,0 +72,4 @@
log.Infof("local !exits\n")
nick, exists = the.Peer.GetContactAttribute(id, attr.GetPeerScope(constants.Name))
if !exists {
log.Infof("peer !exists\n")
sarah прокомментировал(а) 2020-03-30 18:35:36 +00:00
Владелец

and this one

and this one
go/ui/manager.go Устаревшее
@ -85,3 +103,1 @@
return RandomGroupImage(id)
} else {
return profilePicRelativize(pic)
log.Debugln("is group")
sarah прокомментировал(а) 2020-03-30 18:36:02 +00:00
Владелец

remove overly verbose debug statements like this one

remove overly verbose debug statements like this one
go/ui/manager.go Устаревшее
@ -97,1 +135,4 @@
func getPicturePath(pic *image) string {
log.Debugf("getPicturePath for %v\n", pic)
if pic.T == TypeImageDistro {
sarah прокомментировал(а) 2020-03-30 18:36:29 +00:00
Владелец

should be a switch statement with a default

should be a switch statement with a default
buildbot прокомментировал(а) 2020-03-30 21:20:29 +00:00
Участник

Drone Build Status: success

https://build.openprivacy.ca/cwtch.im/ui/511

Drone Build Status: success https://build.openprivacy.ca/cwtch.im/ui/511
sarah закрыл этот запрос на слияние 2020-03-31 19:06:38 +00:00
Это репозиторий в архиве. Вы не можете комментировать запросы на слияние.
Нет рецензентов
Нет этапа
Нет назначенных лиц
3 участников
Срок выполнения
Срок действия недействителен или находится за пределами допустимого диапазона. Пожалуйста, используйте формат 'гггг-мм-дд'.

Срок выполнения не установлен.

Зависимости

Зависимостей нет.

Ссылка: cwtch.im/ui#272
Описание отсутствует.