Skip to content

Conversation

@0xNgoo
Copy link
Contributor

@0xNgoo 0xNgoo commented Jun 9, 2025

📌 Pull Request Title

Description

Implemented A secure and immutable ink! smart contract for issuing verifiable educational
certificates on Polkadot/Substrate-based blockchains.

Related Issues

Closes #91

Key Functions

Admin Functions

  • issue_certificate() - Issue new certificates to students
  • add_admin() - Add new administrators (owner only)
  • remove_admin() - Remove administrators (owner only)

Public Query Functions

  • verify_certificate() - Get certificate details by ID
  • has_certificate() - Check if student has certificate for specific course
  • get_student_certificates() - Get all certificates for a student
  • get_certificate_id() - Get certificate ID for student/course combination
  • is_admin() - Check if account is an admin
  • get_owner() - Get contract owner address
  • get_total_certificates() - Get total number of certificates issued

How to Test

cargo test

Screenshots (if applicable)

Screenshot 2025-06-09 at 6 15 47 AM

Checklist

  • My code follows the project's coding style.
  • I have tested these changes locally.
  • Documentation has been updated where necessary.

Copy link
Contributor

@portableDD portableDD left a comment

Choose a reason for hiding this comment

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

you delete two files, can i know why is the??

to me it look like it only readme you added. you did not do the task assigned to you

@0xNgoo
Copy link
Contributor Author

0xNgoo commented Jun 10, 2025

They were not needed.

Please, load difference to see code difference or pull code. and review. There were actual changes and i have implemented everything stated in the issue.

You can see the test result also

@0xNgoo
Copy link
Contributor Author

0xNgoo commented Jun 10, 2025

This.
Screenshot 2025-06-10 at 12 15 13 PM

Please review code properly sir.

Copy link
Contributor

@portableDD portableDD left a comment

Choose a reason for hiding this comment

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

no comment, files changes did not actual show what you did aside from readme.

@portableDD portableDD merged commit e618335 into Nexacore-Org:main Jun 10, 2025
3 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.

Implement Certificate Smart Contract

2 participants