documenso icon indicating copy to clipboard operation
documenso copied to clipboard

docs: add missing translate:compile step to setup guid (fixes #1761)

Open Harishraju04 opened this issue 2 months ago • 5 comments

Description

Added a missing setup step to run npm run translate:compile before npm run dev. This ensures translation files are compiled correctly, fixing issues with SignUp and Login elements not working.

Related Issue

Fixes #1761

Changes Made

  • Change 1 - i have added this line to the README.md file : 6. Run npm run translate:compile in the root dierctory to compile lingui

Checklist

  • [x] I have tested these changes locally and they work as expected.
  • [x] I have updated the documentation to reflect these changes, if applicable.
  • [x] I have followed the project's coding style guidelines.
  • [x] I have addressed the code review feedback from the previous submission, if applicable.

Harishraju04 avatar Oct 29 '25 06:10 Harishraju04

@Harishraju04 is attempting to deploy a commit to the Documenso Team Team on Vercel.

A member of the Team first needs to authorize it.

vercel[bot] avatar Oct 29 '25 06:10 vercel[bot]

[!IMPORTANT]

Review skipped

Auto reviews are limited based on label configuration.

:label: Required labels (at least one) (1)
  • coderabbit

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

[!NOTE]

Other AI code review bot(s) detected

CodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • [ ] Create PR with unit tests
  • [ ] Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

coderabbitai[bot] avatar Oct 29 '25 06:10 coderabbitai[bot]

Hey there, it looks like you haven't accepted our contributor license agreement yet. In order for us to accept your pull request we ask that you please fill out the CLA:

https://documen.so/cla

Mythie avatar Oct 29 '25 06:10 Mythie

hey @Mythie i have accepted the contributor license agreement.

Harishraju04 avatar Oct 29 '25 06:10 Harishraju04

Wouldn't it be better if the npm run translate command was executed with every new commit? In that case, translators in Crowdin would have new strings to translate and each copy of repository would contain latest file.

mKoonrad avatar Oct 30 '25 19:10 mKoonrad