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

Pedro/tok 31 cambiar sliders por los de shadcn #80

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

pedroinciarte
Copy link
Collaborator

@pedroinciarte pedroinciarte commented Dec 6, 2023

Changes description

Tuve que crear arrays con los valores del hue, chroma y tone porque el Slider recibe como parámetro un number[].
Hice esos cambios y también le agregue al slider una variable background.

Checklist before requesting a review

  • The PR has one clear objective.
  • The PR title briefly describes this objective.
  • Any extra detail needed is on the PRs description.
  • The code is clean and production-ready.
    • Folders, files, functions, and variables are named correctly.
    • No innecesary console.log.
    • Code is correctly fromatted.
  • The PR is optimized for the reviewer, it has small of lines of code.
  • The PR is connected to a Linear Issue.

Pull Request Playbook: https://www.notion.so/Pull-Request-Playbook-959be8e9b41e45e9b2901f0eac7ae7a7?pvs=4

Copy link

vercel bot commented Dec 6, 2023

@pedroinciarte is attempting to deploy a commit to the Tokenbase Team on Vercel.

To accomplish this, @pedroinciarte needs to request access to the Team.

Afterwards, an owner of the Team is required to accept their membership request.

If you're already a member of the respective Vercel Team, make sure that your Personal Vercel Account is connected to your GitHub account.

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.

1 participant