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

ease contributions of translations #1189

Merged
merged 3 commits into from
Mar 19, 2023
Merged

Conversation

samuelstroschein
Copy link
Contributor

@samuelstroschein samuelstroschein commented Feb 14, 2023

Hi there,

This PR makes it easier to a) contribute translations and b) reduces the maintenance effort of the custom contribution solution by adding github.com/inlang.

I saw your online editor https://listmonk.app/i18n/ and corresponding contributions guidelines. Super nice that you have a visual UI in place! Contributing directly from the UI is not possible though, and you need to maintain (?) the editor. I am working on a git-based editor that aims to help OSS projects to get contributions for translations. I thought that this PR might help you already.

You can try the editor in my fork https://inlang.com/editor/github.com/samuelstroschein/listmonk

CleanShot 2023-02-14 at 16 33 46@2x

@samuelstroschein
Copy link
Contributor Author

I forgot, your editor has a feature that ours does not have yet: Showing missing translations. A PR for "eslint for translations" is pending opral/monorepo#366

@samuelstroschein samuelstroschein marked this pull request as ready for review February 17, 2023 12:06
@knadh
Copy link
Owner

knadh commented Mar 6, 2023

Hi @samuelstroschein. Sorry for the delay in responding.

I was just checking https://inlang.com/editor/github.com/samuelstroschein/listmonk and it renders a blank page.

@samuelstroschein
Copy link
Contributor Author

@knadh Bug is filed opral/monorepo#427. Will hopefully be fixed today. I'll reply again once it's fixed. Thanks for the ping!

@samuelstroschein
Copy link
Contributor Author

@knadh Bug has been fixed! :)

PS for a better understanding of what inlang provides, opral/monorepo#416 (reply in thread) should provide more context

@knadh knadh merged commit 4181d8a into knadh:master Mar 19, 2023
samuelstroschein added a commit to opral/monorepo that referenced this pull request Mar 19, 2023
samuelstroschein added a commit to opral/monorepo that referenced this pull request Mar 27, 2023
commit b8cdf3f
Author: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Date:   Mon Mar 27 18:43:22 2023 +0200

    Version Packages (#456)

    Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

commit e9e9ce5
Author: Samuel Stroschein <[email protected]>
Date:   Mon Mar 27 18:35:32 2023 +0200

    refactor: simplify core/lint (#459)

    * refactor: remove setup and teardown

    * work in progress commit

    * refactor: remove the concept of passing booleans to the lint level

    * fix: propagation of context

    Context can't be passed around from nodes to childs without adjusting the target and reference nodes.

    * chore: simplify type

    The generic output was unused.

    * test: slightly more realistic test

    * refactor: remove context

    Redundant concept.

    * init alternative

    * refactor: define setup function in `LintRule`

    #459 (comment)

    * clean up

    * refactor: remove rule collections 😢

    We should wait for user demand to nail a solution.

    * update readme

    * refactor: remove unusued utilities

    The `utilities` lib is intended for external usage. Furthermore, most utilties were not used.

    * add: typesafety to createLintRule settings

    * chore: fix type imports

    * chore: fix typescript compilation

    * chore: remove test utilities export

    The utilities like `createMessage` should likely be part of `ast`. Exposing them only for lint seems a short term solution. But, if we expose those utiltites in lint now, we can't remove them in the future. Thus, remove for now.

    * refactor: return errors to enable apps to handle errors

    Encountered numerous problems because `lint` implicitly catches errors. That behavior makes sense. Just because one rule is throwing, the linting process shouldn't abort.

    However, debugging became harder and consuming apps like the editor or CLI will never now if a linting process failed.

    * fix typo

    Co-authored-by: Hofer Ivan <[email protected]>

    * fix typo

    Co-authored-by: Hofer Ivan <[email protected]>

    * show example lint rule with settings

    * chore: remove todo

    * fix: only pass message to a node

    * chore: add comment to remove node types

    * refactor: narrow settings type

    * refactor: createLintRule only uses args

    #459 (comment)

    * chore: don't use node globals

    * refactor: remove node dependency entirely

    node fs imports don't work in core

    * refactor: remove node dependency entirely

    node fs imports don't work in core

    * refactor: remove node dependency entirely

    node fs imports don't work in core

    * fix: pass correct args

    * Create .changeset/hungry-carpets-build.md

    ---------

    Co-authored-by: Hofer Ivan <[email protected]>

commit f87f300
Author: Boian Ivanov <[email protected]>
Date:   Mon Mar 27 15:28:45 2023 +0100

    chore: linting errors (#469)

    * fix: linting errors

    * chore: add changeset

commit 2d3c8cc
Author: NiklasBuchfink <[email protected]>
Date:   Mon Mar 27 08:31:09 2023 +0000

    refactor: svg logo in website layout

commit b13dfa8
Author: Ivan Hofer <[email protected]>
Date:   Sat Mar 25 11:07:37 2023 +0100

    run npm i

commit 704c83d
Author: Hofer Ivan <[email protected]>
Date:   Sat Mar 25 10:16:04 2023 +0100

    add sdk-js RFC (#460)

commit 4992681
Author: Niklas Buchfink <[email protected]>
Date:   Thu Mar 23 19:12:45 2023 +0100

    Editor condense textarea (#461)

commit 277e5ae
Author: NiklasBuchfink <[email protected]>
Date:   Tue Mar 21 10:05:51 2023 +0000

    add: discord invite to footer

commit 0a2de81
Author: NiklasBuchfink <[email protected]>
Date:   Mon Mar 20 16:13:46 2023 +0000

    add discord invite icon to header

commit a0b85eb
Author: Samuel Stroschein <[email protected]>
Date:   Mon Mar 20 12:52:43 2023 +0100

    refactor: improve core/lint (#453)

    * refactor: use minimum config schema, remove env from lint

    * chore: format fix

    * refactor: only expose 2 query functions

    * chore: add changeset

    * chore: remove unnecessary optional chaining

    * refactor: don't return undefined in lint

    #453 (comment)

    * refactor: immutable lint

commit e3fb6a3
Author: NiklasBuchfink <[email protected]>
Date:   Mon Mar 20 10:14:06 2023 +0000

    fix contact mail in footer

commit 1a6ccec
Author: Nikita <[email protected]>
Date:   Mon Mar 20 02:25:45 2023 +0100

    add: do shallow clone  (#454)

    * do git clone with depth 1

    * add git shallow clone for fast UI response with fetching rest of history in background

    * comment

    * fix prettier

commit 6c8a576
Author: Samuel Stroschein <[email protected]>
Date:   Sun Mar 19 12:53:22 2023 +0100

    Update repositories.ts

    knadh/listmonk#1189

commit 6dbdfc7
Author: Robin Bühler <[email protected]>
Date:   Fri Mar 17 14:32:45 2023 +0000

    chore: add ide-extension publication to ci-cd env

commit a4581da
Author: Robin Bühler <[email protected]>
Date:   Fri Mar 17 14:23:39 2023 +0000

    chroe: change vsce token env

commit 53423d6
Author: Robin Bühler <[email protected]>
Date:   Fri Mar 17 13:57:41 2023 +0000

    chore: run vsce with npx

commit 042bfab
Author: Robin Bühler <[email protected]>
Date:   Fri Mar 17 13:51:23 2023 +0000

    chore: enhance publis-ide-extension-to-marketplace.yml

commit bdbbf8e
Author: Robin Bühler <[email protected]>
Date:   Fri Mar 17 13:38:49 2023 +0000

    chore: update package-lock.json

commit 8f73244
Author: Robin Bühler <[email protected]>
Date:   Fri Mar 17 14:35:56 2023 +0100

    Set up continuous deployment for ide-extension (#446)

    * chore: enable tests

    * chore: name core publish flow

    * chore: add publish workflow for ide-extension

    * chore: format

    * chore: remove ovsx and adjust vsce token name

    * chore: enhance package naming

    * chore: enhance prepare package script

commit 2983569
Author: Nils Jacobsen <[email protected]>
Date:   Thu Mar 16 07:36:22 2023 +0100

    Text fixes (#449)

    * index changes after initial pull

    * feat: add folderstructure for sections #416

    * setup landingpage structure

    * feat: add Section Wrapper for landingpage #416

    * Test second hero

    * feat: add slate colors to ds #416

    * #416 relocate landingpage files in website/index

    * feat: add bg grid component wrapper #416

    * #416 add button & navigation

    * fix button type error

    * feat: add hero layout #416

    * feat: add credibilty section and editor #416

    * feat: add vscode extension section and cli section and footer #416

    * #416 add useNagition to Button component

    * #416 add custom header for landingpage

    * feat: add responsive layout for footer, hero, editor and vscode #316

    * #416 add chevron to Buttons

    * #416 add responsive GetStarted section

    * #416 add functions to Button

    * #416 add signal to toggle CLI section

    * feat: add responsive cli section

    * #416 fix menu background

    * #416 optimize Lighthouse score

    * #416 fix nav for all pages

    * #416 update wording

    * highlighting for copytext

    * feat: add product videos #416

    * fix text & text-width

    * body text width

    * fix: editor layout #416

    * fix: padding on editor #416

    * fix: changes bellong to suggestions of code review #416

    * fix: small wording fixes #416

    ---------

    Co-authored-by: NiklasBuchfink <[email protected]>

commit d913ef2
Author: Nils Jacobsen <[email protected]>
Date:   Tue Mar 14 07:04:01 2023 +0100

    New landing page (#444)

    * index changes after initial pull

    * feat: add folderstructure for sections #416

    * setup landingpage structure

    * feat: add Section Wrapper for landingpage #416

    * Test second hero

    * feat: add slate colors to ds #416

    * #416 relocate landingpage files in website/index

    * feat: add bg grid component wrapper #416

    * #416 add button & navigation

    * fix button type error

    * feat: add hero layout #416

    * feat: add credibilty section and editor #416

    * feat: add vscode extension section and cli section and footer #416

    * #416 add useNagition to Button component

    * #416 add custom header for landingpage

    * feat: add responsive layout for footer, hero, editor and vscode #316

    * #416 add chevron to Buttons

    * #416 add responsive GetStarted section

    * #416 add functions to Button

    * #416 add signal to toggle CLI section

    * feat: add responsive cli section

    * #416 fix menu background

    * #416 optimize Lighthouse score

    * #416 fix nav for all pages

    * #416 update wording

    * highlighting for copytext

    * feat: add product videos #416

    * fix text & text-width

    * body text width

    * fix: editor layout #416

    * fix: padding on editor #416

    * fix: changes bellong to suggestions of code review #416

    ---------

    Co-authored-by: NiklasBuchfink <[email protected]>

commit 6500378
Author: NiklasBuchfink <[email protected]>
Date:   Thu Mar 9 15:30:09 2023 +0000

commit e75807a
Author: Samuel Stroschein <[email protected]>
Date:   Thu Mar 9 13:49:06 2023 +0000

    chore: update turborepo

    Avoiding exec runtime error in docker container.

commit 9ddd3da
Author: Jannes Blobel <[email protected]>
Date:   Thu Mar 9 00:10:30 2023 +0100

    Update config.md
felixhaeberle pushed a commit to opral/monorepo that referenced this pull request Apr 4, 2023
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.

2 participants