[FEAT] Enhance gsheet translations pull python script
ngiangre opened this issue · 6 comments
⚠️ IMPORTANT: Please fill out this template to give us as much information as possible to consider/implement the feature.
Prerequisites
- check this box if you have completed the following:
- Reviewed the contributing guidelines and support files
- Reviewed the README file for the repository you are working in
- Searched for relevant instructions on our Discord server
- Searched the issues of the repository you are working in to make sure one was not already filed
Summary
See this PR for more comments #691.
We needed to merge the PR to move on to integration of translations, but there's more work in enhancing the python script. Broadly,
- we can provide better docs on how to create a google credentials file,
- make more modular and distinct functionality such as only doing depunctuating in one method versus duplicating efforts in multiple methods, and
- making the code overall less prone to bugs and errors from small changes in the gsheets
Motivation
Possible Alternatives
Additional Context
Could you show where there's multiple instances of depunctuation? I haven't touched this code too much, but definitely agree with your second point specifically. I'd be happy to make another issue out of that bullet point especially since that would be a great first issue, but I just want to be sure I have enough context 👍
Could we make another issue for updating documentation as well? I could, but I'm not sure of everything that needs to go into the documentation here.
Other than that, I think we can close this issue. While it's definitely worthwhile mentioning that the code should be less prone to bugs, it's hard to quantify that into actionable items 👍
Yeah I get that. Is there a way to make related issues that are like subissues? Would it just be varying the issue title?
Within the issue you can reference this one just using #702 and it'll reference them in the conversation I believe, although I'm not sure that a new issue would need to be inherently linked just since we're ideally going to be having more discrete action items for the documentation issue