Skip to content

made two new endpoints for user and one new endpoint for staff - #31

Merged
AkibDa merged 1 commit into
GreenPlateByCodeNewbies:mainfrom
AkibDa:main
Jan 17, 2026
Merged

AkibDa merged 1 commit into
GreenPlateByCodeNewbies:mainfrom
AkibDa:main

Conversation

@AkibDa

@AkibDa AkibDa commented Jan 17, 2026

Copy link
Copy Markdown
Member

now the users can update their username which the staff can see when receiving the order, when a student pays for a order they will get an otp which can later be verified by the staff

Comment thread app/user.py
)

# 4️⃣ Update order
pickup_code = str(random.randint(1000, 9999))

Check notice

Code scanning / Bandit (reported by Codacy)

Standard pseudo-random generators are not suitable for security/cryptographic purposes. Note

Standard pseudo-random generators are not suitable for security/cryptographic purposes.

@github-advanced-security github-advanced-security AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Prospector (reported by Codacy) found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.

@github-advanced-security github-advanced-security AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pylintpython3 (reported by Codacy) found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.

@github-advanced-security github-advanced-security AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pylint (reported by Codacy) found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.

@AkibDa
AkibDa merged commit 5f9ca80 into GreenPlateByCodeNewbies:main Jan 17, 2026
8 of 9 checks passed
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.

2 participants