Skip to content

Conversation

Axorax
Copy link
Contributor

@Axorax Axorax commented Dec 31, 2024

Description

Type of Change

  • ✨ New snippet
  • πŸ›  Improvement to an existing snippet
  • 🐞 Bug fix
  • πŸ“– Documentation update
  • πŸ”§ Other (please describe):

Checklist

  • I have tested my code and verified it works as expected.
  • My code follows the style and contribution guidelines of this project.
  • Comments are added where necessary for clarity.
  • Documentation has been updated (if applicable).
  • There are no new warnings or errors from my changes.

Related Issues

Closes #

Additional Context

Screenshots (Optional)

Click to view screenshots

Copy link

netlify bot commented Dec 31, 2024

βœ… Deploy Preview for quicksnip ready!

Name Link
πŸ”¨ Latest commit 3839ee7
πŸ” Latest deploy log https://app.netlify.com/sites/quicksnip/deploys/677406903fd33e0009632f97
😎 Deploy Preview https://deploy-preview-60--quicksnip.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.

@Axorax
Copy link
Contributor Author

Axorax commented Dec 31, 2024

closed previous one due to merge conflicts. I also removed the repeated ones and resolved conflicts.

It's a bit hard to see what's already added because the site search doesn't work.

@Mathys-Gasnier
Copy link
Collaborator

closed previous one due to merge conflicts. I also removed the repeated ones and resolved conflicts.

It's a bit hard to see what's already added because the site search doesn't work.

You don't need to close, and then reopen to resolve merge conflicts, you can merge main into your fork/branch or use the tools github provides.
Please avoid closing and re openning issues like this, it makes reviewing a pain.

@Mathys-Gasnier Mathys-Gasnier added update needed Code needs to be updated. Snippets labels Dec 31, 2024
@Axorax
Copy link
Contributor Author

Axorax commented Dec 31, 2024

closed previous one due to merge conflicts. I also removed the repeated ones and resolved conflicts.
It's a bit hard to see what's already added because the site search doesn't work.

You don't need to close, and then reopen to resolve merge conflicts, you can merge main into your fork/branch or use the tools github provides. Please avoid closing and re openning issues like this, it makes reviewing a pain.

Sorry. I didn't close it myself. I went to fix the conflict and the issue closed by itself.

@Axorax
Copy link
Contributor Author

Axorax commented Dec 31, 2024

hopefully it's all good now

@@ -910,33 +910,6 @@
"tags": ["javascript", "function", "compose", "utility"],
"author": "axorax"
},
{
"title": "Rate Limit Function",
Copy link
Collaborator

Choose a reason for hiding this comment

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

The Random string is a duplicate, this one was fine... Sorry i should have included the name of the snippet

@Axorax
Copy link
Contributor Author

Axorax commented Dec 31, 2024

Done!

@Mathys-Gasnier Mathys-Gasnier merged commit c8335e2 into quicksnip-dev:main Dec 31, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Snippets update needed Code needs to be updated.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants