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

feat!: upgrade generate_bulk_certificate_exceptions to drf ( 26 ) #35577

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

awais786
Copy link
Contributor

@awais786 awais786 commented Oct 2, 2024

Issue

Verify via postman

  1. pass the url in postman using post.
  2. upload the csv file. ( form-data: key= students_list and value = csvfile. )

expected result

{
    "general_errors": [],
    "row_errors": {
        "data_format_error": [],
        "user_not_exist": [],
        "user_already_allowlisted": [],
        "user_not_enrolled": [],
        "user_on_certificate_invalidation_list": []
    },
    "success": [
        "user \"admin\" in row# 1",
        "user \"awais_3\" in row# 2"
    ]
}

Verify via instructor Dashboard

  1. Open this Page
  2. Go to Bulk Exceptions part and choose the file and press Add to exception list.

Expected result
It will show the message 2 learners were successfully added to exception list

Screenshot 2024-10-05 at 3 50 15 PM

demo csv file attached = students_list.csv

entries will appear in this table http://local.edly.io:8000/admin/certificates/certificateallowlist/

@awais786 awais786 changed the title Generate bulk certificate exceptions to drf feat!: upgrade generate_bulk_certificate_exceptions to drf ( 26 ) Oct 4, 2024
@awais786 awais786 marked this pull request as ready for review October 5, 2024 10:59
@feanil feanil force-pushed the generate_bulk_certificate_exceptions-to-drf branch from 9e312c8 to 1541862 Compare November 14, 2024 16:14
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