search

glossarist / glossarist-desktop

Glossarist Desktop
https://www.glossarist.org
GNU General Public License v3.0
4 stars 6 forks source link

(Was URGENT) Unable to push accepted CR changes #147

Closed ronaldtse closed 4 years ago

ronaldtse commented 4 years ago

In https://github.com/geolexica/isotc211-glossary, the TMG convener was able to accept + resolve, and see the new translated term, but the data is unable to upload to the repository.

In a screen share it is confirmed that the CR is accepted, the local repository works (the new term is accepted and showing in List view).

ronaldtse commented 4 years ago

UPDATE: a GitHub auth token may not have permissions to push data? Problem is solved for now by using password authentication.

Maybe we need some interface to test whether the access credentials work? e.g. push a new branch, and delete it.

strogonoff commented 4 years ago

UPDATE: a GitHub auth token may not have permissions to push data? Problem is solved for now by using password authentication.

Maybe we need some interface to test whether the access credentials work? e.g. push a new branch, and delete it.

A token with “repo” scope is enough, unless there was a recent change in GitHub logic.

strogonoff commented 4 years ago

I believe these things should be filed in https://github.com/geolexica/isotc211-glossary-support/issues, as often (like in this case) they are issues outside of Glossarist Desktop but issues pertaining to infrastructure/integration

strogonoff commented 4 years ago

Moved to https://github.com/geolexica/isotc211-glossary-support/issues/2