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

Feature/seed richer badges #162

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from

Conversation

berkes
Copy link
Contributor

@berkes berkes commented Dec 10, 2024

This adds some extra and richer data to the seeded badges.

So that local development has some more data and attributes to use out of the box.

It improves:

  • Added markdown formatting to the badge_class_description to make it more structured and readable.
  • Updated criteria_text to provide a clearer description of the requirements for earning the badge.

It adds:

  • Introduced quality_assurance_description, quality_assurance_name, and quality_assurance_url attributes for half of the badge classes to include quality assurance information.
  • Added assessment_type attribute to half of the badge classes, with predefined types such as "testing" and "application of a skill".
  • Set assessment_supervised to True for half of the badge classes with assessment_type set to "testing".
  • Set identity_checked to True for half of the badge classes that have assessment_supervised set to True.

@berkes berkes requested a review from Iso5786 December 10, 2024 17:11
@berkes berkes self-assigned this Dec 10, 2024
Wiping the database is:
- slow. It adds over a minute to the boot process of a docker compose up
- cumbersome. We start with a blank slate after every boot, which means
  issued badges, accounts, logins, etc all are gone.

Ideally the blank state is useful in itself, but it's lacking data and
setup currently. So after booting we need quite a few manual tasks each
time, to get relevant badges into users' backpack.

This feature was commented out in the commit:

  commit 2916276
  Author: Okke Harsta <[email protected]>
  Date:   Wed Aug 18 14:24:39 2021 +0200

      Temporary comment to fix deployment

  Δ apps/mainsite/management/commands/seed.py

But, as "temporary fixes" tend to do, it remained in, and now causes
work and delays when developing.

We also add a persistent volume to docker-compose so that the data for
the mysql database is stored outside of the container and can live
across container lifecycles.
@berkes berkes force-pushed the feature/seed-richer-badges branch from c6512ea to 1f12a4c Compare December 17, 2024 09:08
All BadgeClasses in Medicine get alignment-items:
  - EQF
  - ECTS
This user is often used in demo's and in tests.
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