We can do just with two buttons and changing the names.

it turns out that "full path" is still a relative path, and therefore
not allowed as a GOPATH.

Add to 'make generate_locales' the generation of a
'transifex/messages.json' that will be automatically pulled by transifex
for translations.

To incorporate translations from transifex into our project we'll need
to manually download the selected translations and convert them with the
transifex program into gotext format.

-Resolves: #73

this adds upstream PR #74

In case of SNAP environment we need to execute the
/snap/bin/app.launcher and not os.Args[0].

- Resolves: #82

This commit includes a hack to prioritize giraffe, a new riseup gateway
in europe to reduce the load in the existing node.

- Resolves: #81

setting TMPDIR attempted to end the noisy error message about using TMP
producing non readable messages.

however, this produces invisible icons in bionic.

See leap/riseup_vpn#44

after
https://github.com/getlantern/systray/commit/5fb0feca3c0677e9fa31e579ff69631f49a379a2
was merged, we don't need to keep using the vendoring of systray.

there is some flickering with the stop/start/cancel menu items
chening their orderning that still needs to be solved.

- Resolves: #75

When there is a pid file from a previous crash, in some situations the
pid has being reused by the operative system for other processes.
Let's check that the exectuable name maches the name of the systray
binary.

- Resolves: #50

- Resolves: #26

Kali Kaneko authored 6 years ago and Kali Kaneko committed 6 years ago

- Resolves: #64

Let's not report 'bitmaskd is not running' in standalone mode.

- Resolves: #35

Kali Kaneko authored 6 years ago and Kali Kaneko committed 6 years ago

if we're running from snap, we take the version string from a file in
the snap folder.

the version.sh script is executed during the creation of the snap, and
is expected to write a version.txt file in the snap/ folder.

Stop generating one file in /tmp for each icon change. Let's move the
clean up to the set icon instead of waiting for the quit clean up.
Unlink will remove the file from /tmp, but app_indicator will hold a
descriptor until it finishes with it.

This is a cherry-pick from a pull-request upstream:
https://github.com/getlantern/systray/pull/63

- Resolves: #60

Go time library returns the timezone offset in seconds and we where
using it as hours.

- Resolves: #72

see the dockerfile in riseup_vpn/docker, that's uploaded to leap's
docker registry.

Kali Kaneko authored 6 years ago and Kali Kaneko committed 6 years ago

this is a workaround for riseup_vpn#28

Don't trust that the status will be reported by openvpn/bitmaskd
correctly. Let's do polling every 5 seconds.

Let's change the systray state as soon as the user clicks on it, instead
of waiting for openvpn to notify the status change.