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

fix: [oss.gg] solves #7920 #7962

Merged
merged 1 commit into from
Oct 22, 2024
Merged

Conversation

sateshcharan
Copy link
Contributor

@sateshcharan sateshcharan commented Oct 22, 2024

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 PR enhances the readability and structure of the design challenge instructions in the "3-create-custom-interfact-theme-20.md" file.

  • Added Figma link to the main design file for easy access to the original theme
  • Improved formatting with line breaks for better visual separation of sections
  • Updated example schema for submissions, providing clearer guidance for participants
  • Corrected typo in filename from "interfact" to "interface" for accuracy

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


Please follow the following schema:

---

» 05-April-2024 by YOUR oss.gg HANDLE » Figma Link: https://link.to/content
» 05-April-2024 by YOUR oss.gg HANDLE] Figma Link: https://www.figma.com/design/xt8O9mFeLl46C5InWwoMrN/Twenty?t=YIFyswta6Xf6sSYK-0
Copy link
Contributor

Choose a reason for hiding this comment

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

syntax: Remove extra square bracket after 'HANDLE'

Suggested change
» 05-April-2024 by YOUR oss.gg HANDLE] Figma Link: https://www.figma.com/design/xt8O9mFeLl46C5InWwoMrN/Twenty?t=YIFyswta6Xf6sSYK-0
» 05-April-2024 by YOUR oss.gg HANDLE Figma Link: https://www.figma.com/design/xt8O9mFeLl46C5InWwoMrN/Twenty?t=YIFyswta6Xf6sSYK-0

@@ -18,4 +19,4 @@ Your turn 👇

» 02-October-2024 by [yourhandle](https://oss.gg/yourhandle) Figma Link: [Figma](https://twenty.com/)
Copy link
Contributor

Choose a reason for hiding this comment

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

style: Update example link to match the new Figma link format

Suggested change
» 02-October-2024 by [yourhandle](https://oss.gg/yourhandle) Figma Link: [Figma](https://twenty.com/)
» 02-October-2024 by [yourhandle](https://oss.gg/yourhandle) Figma Link: [Figma](https://www.figma.com/design/xt8O9mFeLl46C5InWwoMrN/Twenty?t=YIFyswta6Xf6sSYK-0)

@Bonapara
Copy link
Member

/award 50

Copy link

oss-gg bot commented Oct 22, 2024

Awarding sateshcharan: 50 points 🕹️ Well done! Check out your new contribution on oss.gg/sateshcharan

@Bonapara Bonapara merged commit 32cf88f into twentyhq:main Oct 22, 2024
16 checks passed
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