Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat: add export with untranslated keys #389

Merged
merged 8 commits into from
Jul 13, 2023

Conversation

vvadymk
Copy link
Contributor

@vvadymk vvadymk commented Jul 12, 2023

This change resolves #388 by adding ability to export untranslated keys

@vercel
Copy link

vercel bot commented Jul 12, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
traduora-docs-co ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 13, 2023 6:02pm

@evereq
Copy link
Member

evereq commented Jul 12, 2023

@evereq evereq self-requested a review July 12, 2023 22:05
@evereq evereq self-assigned this Jul 12, 2023
@evereq evereq added this to the Traduora 1.0 milestone Jul 12, 2023
@evereq
Copy link
Member

evereq commented Jul 12, 2023

@vvadymk seems now some tests are failing and it's related to column names changes? Can you check please https://app.circleci.com/pipelines/github/ever-co/ever-traduora/820/workflows/a4bc7635-91ab-487f-be6a-667373a0d79d/jobs/1166/parallel-runs/0/steps/0-110

@vvadymk
Copy link
Contributor Author

vvadymk commented Jul 12, 2023

Hi @evereq! I have already fixed lint issue. Now I am fixing tests

@evereq
Copy link
Member

evereq commented Jul 13, 2023

@vvadymk hm, does not look like tests are fixed... still complains about incorrect names of columns with that "_" etc.

@vvadymk
Copy link
Contributor Author

vvadymk commented Jul 13, 2023

@evereq I am still in work. I haven' t changed the db columns

@vvadymk
Copy link
Contributor Author

vvadymk commented Jul 13, 2023

where do you see that the problem is in db columns?

@evereq
Copy link
Member

evereq commented Jul 13, 2023

In https://app.circleci.com/pipelines/github/ever-co/ever-traduora/820/workflows/a4bc7635-91ab-487f-be6a-667373a0d79d/jobs/1166/parallel-runs/0/steps/0-110, search for "ER_BAD_FIELD_ERROR: Unknown column 'term.project_id' in 'where clause'"

@vvadymk
Copy link
Contributor Author

vvadymk commented Jul 13, 2023

@evereq I got it. I also have made a migration to pgsql and rename columns a bit from MySQL version. I will fix it now

@vvadymk
Copy link
Contributor Author

vvadymk commented Jul 13, 2023

@evereq If I have a working psql version, how could I suggest my variant?

@evereq
Copy link
Member

evereq commented Jul 13, 2023

I think we just need to support both MySQL and PostgreSQL with TypeORM, i.e. should be switch based on "type" of the DB, but I wonder, what are differences you notice? I.e. why do we need to change column names, it did not work with those we already have? We are very interested to make sure Traduora works well with PostgreSQL same as our other products, will be great if you can help with that. Maybe you can create a separate GitHub issue about PostgreSQL support and let's discuss it there if you are interested? @vvadymk

@vvadymk
Copy link
Contributor Author

vvadymk commented Jul 13, 2023

There was a lot of problems during migration to pgsql. But finally, I have migrated it and it works good. I renamed it because there was conflicts between table and column naming un pgsql

@vvadymk
Copy link
Contributor Author

vvadymk commented Jul 13, 2023

I have fixed problem with this branch and pushed right now

@vvadymk
Copy link
Contributor Author

vvadymk commented Jul 13, 2023

I could create an issue and branch with my pgsql version

@vvadymk
Copy link
Contributor Author

vvadymk commented Jul 13, 2023

@evereq it could be merged. I have fixed all issues.

@evereq evereq merged commit bd24619 into ever-co:develop Jul 13, 2023
csampaio26 pushed a commit to KenbiTech/traduora that referenced this pull request Sep 4, 2023
* feat: add export with untranslated keys

* fix: check with prettier

* fix: fix lint error

* fix: edit e2e tests for untranslated feature

* fix: fix tests

* fix: fix column name in query

* fix: naming of the projectId to camelCase

* fix: fix tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add ability to export untranslated keys
2 participants