Bitmessage
/
transifex-webhook
Archived
2
0
Fork 2
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Transifex_demo #1

Closed
cisosama wants to merge 1 commits from cisosama/transifex-webhook:Transifex_demo into main
pull from: cisosama/transifex-webhook:Transifex_demo
merge into: Bitmessage:main
Conversation 1 Commits 1 Files Changed 17 +804
cisosama commented 2022-12-14 07:42:39 +00:00
First-time contributor
Copy Link

Transifex demo added, now source string is translated into french

Transifex demo added, now source string is translated into french
cisosama added 2 commits 2022-12-14 07:42:39 +00:00
PeterSurda requested changes 2022-12-19 06:49:07 +00:00
PeterSurda left a comment
Owner
  • Sanitize
  • in general correct but incomplete (no requirements.txt, Dockerfile, docker-compose.yml, webhook argument processing)
- Sanitize - in general correct but incomplete (no `requirements.txt`, `Dockerfile`, `docker-compose.yml`, webhook argument processing)
cisosama force-pushed Transifex_demo from bff51a74da to ee0e081983 2022-12-19 13:25:54 +00:00 Compare
cisosama force-pushed Transifex_demo from ee0e081983 to fcade1debb 2022-12-19 13:26:47 +00:00 Compare
cisosama force-pushed Transifex_demo from fcade1debb to 3628348322 2022-12-19 13:55:46 +00:00 Compare
PeterSurda requested changes 2022-12-20 06:59:09 +00:00
Dockerfile Outdated
@ -0,0 +1,5 @@
FROM python:3
ADD requirements.txt /
PeterSurda commented 2022-12-20 06:40:52 +00:00
Owner

I would create a directory for the app, such as /app

I would create a directory for the app, such as `/app`
transifex_demo.py Outdated
@ -0,0 +1,17 @@
"""Transifex basic implementation using python native sdk """
PeterSurda commented 2022-12-20 06:42:20 +00:00
Owner

should be made into a web app (e.g. cherrypy, flask, django, ...)

should be made into a web app (e.g. cherrypy, flask, django, ...)
transifex_demo.py Outdated
@ -0,0 +4,4 @@
from transifex.native import init, tx
from transifex.native.parsing import SourceString
env_path=os.path.join(os.getcwd(),'transifex.env')
PeterSurda commented 2022-12-20 06:44:12 +00:00
Owner

since we're already telling docker-compose to load the transifex.env file, we don't need to do this anymore

since we're already telling docker-compose to load the `transifex.env` file, we don't need to do this anymore
PeterSurda marked this conversation as resolved
cisosama added 1 commit 2022-12-20 14:45:15 +00:00
cisosama force-pushed Transifex_demo from 532389c76e to 1971fb0604 2022-12-21 07:04:47 +00:00 Compare
cisosama force-pushed Transifex_demo from 1971fb0604 to 761d0b7bbb 2022-12-21 10:23:02 +00:00 Compare
cisosama force-pushed Transifex_demo from 761d0b7bbb to ed85f418af 2022-12-21 10:32:30 +00:00 Compare
cisosama force-pushed Transifex_demo from ed85f418af to 9f7384d6c6 2022-12-21 15:04:15 +00:00 Compare
PeterSurda approved these changes 2022-12-27 06:46:52 +00:00
transifex_flask/flask_app/app.py Outdated
@ -0,0 +71,4 @@
response = requests.put("https://www.transifex.com/api/2/project/pybitmessage/resource/pybitmessage/content/",
headers=headers, files={'bitmessage_en.ts': open("src/translations/bitmessage_en.ts", "rb")})
return response
PeterSurda commented 2022-12-27 06:45:57 +00:00
Owner 
@expose
def upload_source()
	queue_async_job(id)
    return ok
    
def async_job(id)
   while not  requests.get url+id
      sleep (1)
      # todo expire after maybe 10 minutes
   transifex_api_upload_resource
``` @expose def upload_source() queue_async_job(id) return ok def async_job(id) while not requests.get url+id sleep (1) # todo expire after maybe 10 minutes transifex_api_upload_resource ```
PeterSurda requested changes 2022-12-27 06:47:16 +00:00
PeterSurda left a comment
Owner

please make the changes required

please make the changes required
cisosama added 2 commits 2022-12-28 15:53:55 +00:00
cisosama force-pushed Transifex_demo from 52cac358b4 to 9cfbadbe45 2022-12-30 06:10:34 +00:00 Compare
cisosama force-pushed Transifex_demo from 9cfbadbe45 to 3052e97d90 2022-12-30 06:13:06 +00:00 Compare
cisosama force-pushed Transifex_demo from 3052e97d90 to 02246b8602 2022-12-30 06:19:25 +00:00 Compare
cisosama force-pushed Transifex_demo from 02246b8602 to 03132afaf7 2022-12-30 06:29:39 +00:00 Compare
PeterSurda commented 2023-01-06 06:33:39 +00:00
Owner
Copy Link

obsolete

obsolete
PeterSurda closed this pull request 2023-01-06 06:33:39 +00:00
This repo is archived. You cannot comment on pull requests.
Reviewers
No reviewers
PeterSurda
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
2 Participants
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: Bitmessage/transifex-webhook#1
No description provided.
Delete Branch "cisosama/transifex-webhook:Transifex_demo"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?