Skip to content
This repository has been archived by the owner on May 14, 2024. It is now read-only.

[FIX] Update egg-j-music-bot.json #1650

Merged
merged 2 commits into from
May 19, 2022
Merged

Conversation

DaFray31
Copy link
Contributor

Token from Discord have length equal to 70 not 64

All Submissions:

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • Did you branch your changes and PR from that branch and not from your master branch?
    • If not, why?:

Changes to an existing Egg:

  1. Have you added an explanation of what your changes do and why you'd like us to include them?
  2. Have you tested your Egg changes?

Token from discord have length equal to 70 not 64
@redthirten
Copy link
Collaborator

Should we just remove the limit all together? My key is 59 in length and yours is apparently 70, so I guess it isn't a set length? I couldn't find any documentation on it.

@docimin
Copy link
Contributor

docimin commented May 16, 2022

Hey @lilkingjr1, that is correct. No length is required, so using "rules": "required|string" would work fine.

@DaFray31
Copy link
Contributor Author

Yeah, sorry for the time I took, that should be correct now

@parkervcp parkervcp merged commit ce7ed22 into pelican-eggs:master May 19, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants