Closed avikalpg closed 1 week ago
vercel[bot]
commented
1 week ago The latest updates on your projects. Learn more about Vercel for Git ↗︎
| Name | Status | Preview | Comments | Updated (UTC) |
|---|---|---|---|---|
| team-monitor-website | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Sep 5, 2024 5:57am |
vibinex-dpu[bot]
commented
1 week ago Relevant users for this PR:
| Contributor Name/Alias | Relevance |
|---|---|
| tapishr | 92.86% |
| avikalpg | 7.14% |
Auto assigning to relevant reviewers.
If you are a relevant reviewer, you can use the Vibinex browser extension to see parts of the PR relevant to you Relevance of the reviewer is calculated based on the git blame information of the PR. To know more, hit us up at contact@vibinex.com.
To change comment and auto-assign settings, go to your Vibinex settings page.
coderabbitai[bot]
commented
1 week ago [!IMPORTANT]
Review skipped
Review was skipped due to path filters
Files ignored due to path filters (1)
* `package-lock.json` is excluded by `!**/package-lock.json`You can disable this status message by setting the
reviews.review_statustofalsein the CodeRabbit configuration file.
The recent updates include enhancements to the CodeWithCopyButton.tsx for improved text handling and user feedback on copy actions. The DockerInstructions.tsx has been simplified by streamlining the command output based on selected conditions. Additionally, the package.json file reflects a minor version increment, indicating small improvements or bug fixes.
| File Path | Change Summary |
|---|---|
| components/setup/CodeWithCopyButton.tsx | Introduced text cleanup functionality, added useEffect for managing copy button state, and implemented a timer for feedback. |
| components/setup/DockerInstructions.tsx | Modified selfHostingCode string template to simplify output by removing unnecessary environment variable settings. |
| package.json | Updated version number from "1.2.12" to "1.2.13." |
🐰 In the garden where code does play,
A button now shines, brightening the day.
Docker commands, once tangled and long,
Are now simple tunes, a sweet, cheerful song.
With each little change, we hop with delight,
Celebrating progress, our future is bright! 🌼
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
vibinex-dpu[bot]
commented
1 week ago Relevant users for this PR:
| Contributor Name/Alias | Relevance |
|---|---|
| tapishr | 81.25% |
| avikalpg | 18.75% |
Auto assigning to relevant reviewers.
If you are a relevant reviewer, you can use the Vibinex browser extension to see parts of the PR relevant to you Relevance of the reviewer is calculated based on the git blame information of the PR. To know more, hit us up at contact@vibinex.com.
To change comment and auto-assign settings, go to your Vibinex settings page.
sonarcloud[bot]
commented
1 week ago 
Quality Gate passedIssues
0 New issues
0 Accepted issues
Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code
vibinex-dpu[bot]
commented
1 week ago Relevant users for this PR:
| Contributor Name/Alias | Relevance | Approval |
|---|---|---|
| tapishr | 81.25% | :white_check_mark: |
| avikalpg | 18.75% | :x: |
Total Coverage for PR: 81.25%
If you are a relevant reviewer, you can use the Vibinex browser extension to see parts of the PR relevant to you Relevance of the reviewer is calculated based on the git blame information of the PR. To know more, hit us up at contact@vibinex.com.
To change comment and auto-assign settings, go to your Vibinex repository settings page.
avikalpg
commented
1 week ago Screenshot would have been helpful
Next time daal dunga. I thought vercel deployment mein dekh paoge aaraam se.
Bug
For multiline code in the docs, when the user copies the contents, they see extra newline characters in the command, which means the command does not work on directly pasting.
Resolution
Modified the
CodeWithCopyButtoncomponent such that it displays the text exactly as it is provided. This way, the default copy functionality also works as expected.Side-effects
As a result, we now don't need the extra newline characters in the docker-instructions component, which were earlier added to make sure that the code is displayed correctly.
Additional feature
Improved the responsiveness of the copy button for subsequent button presses. Earlier, once the button was pressed, the feedback was constantly displayed on the screen. And if the user clicked on the button again, there was no feedback about whether or not was the text correctly copied to the clipboard.