RealDevSquad / RealDevSquad/website-backend

Remove checkIsVerifiedDiscord for discord-actions/nicknames/sync API

Open
#1,989 0 comments 0 reactions 1 assignee View on GitHub

@Pavangbhat is already working on this.

Since Mar 27, 2024.

Dominant language
JavaScript
Stars
74
Forks
276
Avg merge
1d 26m
Merged PRs (30d)
14

Description

Issue Description

Removed Middleware for Reference:

image

This route discord-actions/nicknames/sync is behind a middleware accessing req.userData which would only be available if the call is made through website-dashboard. We need to remove this check as we move these routes to a cronjob

Expected Behavior

The route should also be accessible through cronjob without user details

Current Behavior

req.userData is blocking us from triggering this API through cronjob

Screenshots
Reproducibility
  • This issue is reproducible
  • This issue is not reproducible
Steps to Reproduce
Severity/Priority
  • Critical
  • High
  • Medium
  • Low
Additional Information
Checklist
  • I have read and followed the project's code of conduct.
  • I have searched for similar issues before creating this one.
  • I have provided all the necessary information to understand and reproduce the issue.
  • I am willing to contribute to the resolution of this issue.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.