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: tolgee SDK v6 #793

Merged
merged 17 commits into from
Jan 8, 2025
Merged

feat: tolgee SDK v6 #793

merged 17 commits into from
Jan 8, 2025

Conversation

stepan662
Copy link
Contributor

No description provided.

Copy link

netlify bot commented Dec 28, 2024

Deploy Preview for tolgee-docs ready!

Name Link
🔨 Latest commit d2a0188
🔍 Latest deploy log https://app.netlify.com/sites/tolgee-docs/deploys/677ce9ba975f6e0008f92146
😎 Deploy Preview https://deploy-preview-793--tolgee-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Contributor

@JanCizmar JanCizmar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good job! 👍

  • There is many Grammarly issues, can you please always run it trough Grammarly befor you submit the PR?
  • It looks like you don't have spellcheck in the IDE, do you?
  • Wouldn't be beneficial to create special Next package now, when we release V6 in one bigger update removing the need to the boilerplate code users need to add when working with next?

@stepan662 stepan662 force-pushed the stepangranat/sdk-v6 branch from daaec91 to 7164785 Compare January 6, 2025 12:33
@stepan662 stepan662 requested a review from JanCizmar January 7, 2025 08:34
@stepan662 stepan662 marked this pull request as ready for review January 7, 2025 08:34
Copy link
Contributor

@JanCizmar JanCizmar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since I would like to post this on the Next.js reddit group, I would remove the emojis from the post. Usually on reddit people are pretty sensitive about emojis.

Lately I've posted this link to Kotlin group and it has 100% upvote rate, which I believe is beacuse on Reddit people like "raw" tests which don't look much like self promotion.

https://docs.tolgee.io/blog/2024-12-28-compose-multiplatform-import-export

@stepan662
Copy link
Contributor Author

Ok, done @JanCizmar

@stepan662 stepan662 requested a review from JanCizmar January 7, 2025 08:46
stepan662 added a commit to tolgee/tolgee-js that referenced this pull request Jan 8, 2025
@stepan662 stepan662 merged commit 1623a10 into main Jan 8, 2025
6 checks passed
@stepan662 stepan662 deleted the stepangranat/sdk-v6 branch January 8, 2025 15:12
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