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

side-quest-oss.gg #7938

Closed
wants to merge 1 commit into from
Closed

Conversation

rajeevDewangan
Copy link
Contributor

What side quest are you solving : Write a guide on self-hosting Twenty

Points : 750

Description : Created a comprehensive guide on how to self-host Twenty. Included a list of the most frequent self-hosting errors users encounter when setting up Twenty by reviewing the #Help Discord channel.

Proof :
Screenshot 2024-10-22 125551
link : https://open.substack.com/pub/rajeevdewangan/p/comprehensive-guide-to-self-hosting?r=4lly3x&utm_campaign=post&utm_medium=web&showWelcomeOnShare=true

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

PR Summary

This pull request adds a new file 'side-quest-oss.gg' with minimal content, which does not align with the stated goal of creating a comprehensive self-hosting guide for Twenty.

  • The added file 'side-quest-oss.gg' contains only one line: "Twenty content challenges", not the promised self-hosting guide
  • There's a significant discrepancy between the PR description and the actual changes made
  • The PR references external content (Substack link) instead of adding the guide directly to the repository
  • Existing self-hosting documentation in the repository (/packages/twenty-website/src/content/developers/self-hosting/) is not utilized or updated
  • The PR does not address the stated goal of including frequent self-hosting errors from the Discord #Help channel

1 file(s) reviewed, 1 comment(s)
Edit PR Review Bot Settings | Greptile

@@ -0,0 +1 @@
Twenty content challenges
Copy link
Contributor

Choose a reason for hiding this comment

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

logic: The file content does not match the PR description. Expected a comprehensive guide on self-hosting Twenty.

@charlesBochet
Copy link
Member

@rajeevDewangan thank you! Could you add your blog post to https://github.com/twentyhq/twenty/blob/main/oss-gg/twenty-content-challenges/2-write-blog-post-about-20.md ?

@rajeevDewangan
Copy link
Contributor Author

Added you can check

@charlesBochet
Copy link
Member

I meant in this PR :) But it's fine! I'll close this one and merge the other!

charlesBochet pushed a commit that referenced this pull request Oct 22, 2024
Update oss-gg block post list for
#7938
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants