WIP: Draft: Resolve "Rename Categories to Skills and introduce Skillset" #73

Draft
magicfelix wants to merge 20 commits from 15-rename-categories-to-skills-and-introduce-skillset into master
Member

Closes #15

Closes #15
Author
Member

added 4 commits

  • 19776883 - Migrate all models to ExtensibleModel
  • 7c08ffa6 - Fix some permissions and adapt schema to current core changes and ExtensibleModel
  • 0362c8a4 - Adapt migration to fixed UUID default
  • 04fe63b7 - WIP: Add skillset migration

Compare with previous version

added 4 commits <ul><li>19776883 - Migrate all models to ExtensibleModel</li><li>7c08ffa6 - Fix some permissions and adapt schema to current core changes and ExtensibleModel</li><li>0362c8a4 - Adapt migration to fixed UUID default</li><li>04fe63b7 - WIP: Add skillset migration</li></ul> [Compare with previous version](/Teckids/Indiedact/AlekSIS-App-Paeda/-/merge_requests/31/diffs?diff_id=63167&start_sha=6efc4c0992b42882f9d8fb4c2901f1ab8caf3cf5)
Author
Member

added 1 commit

  • 24d29358 - Migrate category m2m relationships to skillsets

Compare with previous version

added 1 commit <ul><li>24d29358 - Migrate category m2m relationships to skillsets</li></ul> [Compare with previous version](/Teckids/Indiedact/AlekSIS-App-Paeda/-/merge_requests/31/diffs?diff_id=63219&start_sha=04fe63b726b8678323bf536b57c80105fc58d500)
Author
Member

added 2 commits

  • 05eb6ba8 - Rename Category to Skill
  • f0919102 - WIP: Adjust frontend to skillset

Compare with previous version

added 2 commits <ul><li>05eb6ba8 - Rename Category to Skill</li><li>f0919102 - WIP: Adjust frontend to skillset</li></ul> [Compare with previous version](/Teckids/Indiedact/AlekSIS-App-Paeda/-/merge_requests/31/diffs?diff_id=63242&start_sha=24d293582aaad803a85933707383d6a700968ec1)
Author
Member

added 3 commits

  • 70d2d4fa - Adjust frontend to category renaming
  • ebcf7a8c - Add frontend for skillset
  • 173c5cda - Adjust permissions

Compare with previous version

added 3 commits <ul><li>70d2d4fa - Adjust frontend to category renaming</li><li>ebcf7a8c - Add frontend for skillset</li><li>173c5cda - Adjust permissions</li></ul> [Compare with previous version](/Teckids/Indiedact/AlekSIS-App-Paeda/-/merge_requests/31/diffs?diff_id=63357&start_sha=f091910235a76c612a3c4b24fe85d1a7487064b1)
Author
Member

added 13 commits

  • d03a0d64 - 1 commit from branch master
  • d03a0d64...d7bd3fed - 2 earlier commits
  • 5b6b07ed - Adjust frontend to category renaming
  • bd5d4765 - Add frontend for skillset
  • a4df82f4 - Adjust permissions
  • d49a4d69 - Adjust schema and frontend queries to skillset
  • aacffca6 - Add messages for skillset
  • 6b2c755a - Adjust SkillSetCRUDList
  • c6caa264 - Adapt more parts of frontend
  • 0fbf17a1 - Fix skillsets occurence
  • 933fb31b - Fix ProfileCRUDList
  • b5ccc7f3 - Fix Qualification CRUD list

Compare with previous version

added 13 commits <ul><li>d03a0d64 - 1 commit from branch <code>master</code></li><li>d03a0d64...d7bd3fed - 2 earlier commits</li><li>5b6b07ed - Adjust frontend to category renaming</li><li>bd5d4765 - Add frontend for skillset</li><li>a4df82f4 - Adjust permissions</li><li>d49a4d69 - Adjust schema and frontend queries to skillset</li><li>aacffca6 - Add messages for skillset</li><li>6b2c755a - Adjust SkillSetCRUDList</li><li>c6caa264 - Adapt more parts of frontend</li><li>0fbf17a1 - Fix skillsets occurence</li><li>933fb31b - Fix ProfileCRUDList</li><li>b5ccc7f3 - Fix Qualification CRUD list</li></ul> [Compare with previous version](/Teckids/Indiedact/AlekSIS-App-Paeda/-/merge_requests/31/diffs?diff_id=63464&start_sha=173c5cda31cf56a36edb88ddc32d4faa3d0013cc)
Author
Member

added 3 commits

  • 4bd9d591 - Fix editing profiles
  • 9408d26a - Fix subset relationship
  • c8396b00 - Fix skillset selects

Compare with previous version

added 3 commits <ul><li>4bd9d591 - Fix editing profiles</li><li>9408d26a - Fix subset relationship</li><li>c8396b00 - Fix skillset selects</li></ul> [Compare with previous version](/Teckids/Indiedact/AlekSIS-App-Paeda/-/merge_requests/31/diffs?diff_id=63466&start_sha=b5ccc7f35a3b87383954bbc0c96aa08b73a06a0a)
Author
Member

added 1 commit

Compare with previous version

added 1 commit <ul><li>ef928dc0 - Fix skillset selects</li></ul> [Compare with previous version](/Teckids/Indiedact/AlekSIS-App-Paeda/-/merge_requests/31/diffs?diff_id=63482&start_sha=c8396b00bb5a77265f15d642031695d0914f4682)
Author
Member

added 1 commit

Compare with previous version

added 1 commit <ul><li>c8df59b5 - Reformat</li></ul> [Compare with previous version](/Teckids/Indiedact/AlekSIS-App-Paeda/-/merge_requests/31/diffs?diff_id=63486&start_sha=ef928dc0afbe22f480b69037f594fd2ed9fe0873)
Author
Member

added 1 commit

Compare with previous version

added 1 commit <ul><li>0ef3f7ae - Fix skillset migration</li></ul> [Compare with previous version](/Teckids/Indiedact/AlekSIS-App-Paeda/-/merge_requests/31/diffs?diff_id=63501&start_sha=c8df59b5f1623fad9ea8a04287e8ba0450c46df8)
Author
Member

marked this merge request as ready

marked this merge request as **ready**
Author
Member

assigned to @nik

assigned to @nik
Author
Member

added 10 commits

  • 0ef3f7ae...c8db6a5f - 9 commits from branch master
  • 6386394b - Merge branch 'master' into 15-rename-categories-to-skills-and-introduce-skillset

Compare with previous version

added 10 commits <ul><li>0ef3f7ae...c8db6a5f - 9 commits from branch <code>master</code></li><li>6386394b - Merge branch &#39;master&#39; into 15-rename-categories-to-skills-and-introduce-skillset</li></ul> [Compare with previous version](/Teckids/Indiedact/AlekSIS-App-Paeda/-/merge_requests/31/diffs?diff_id=71701&start_sha=0ef3f7ae76ca300a5ac7fc9f9208b05ee388e9b8)
Owner

marked this merge request as draft

marked this merge request as **draft**
Owner

@magicfelix What's the status of this?

@magicfelix What's the status of this?
WIP: Make some adjustments
Some checks failed
CI / lock (push) Successful in 54s
CI / lint (push) Failing after 45s
CI / test (push) Failing after 1m9s
CI / tests-done (push) Failing after 5s
CI / build (push) Successful in 1m34s
CI / publish (push) Has been skipped
f2a3185a8d
magicfelix force-pushed 15-rename-categories-to-skills-and-introduce-skillset from f2a3185a8d
Some checks failed
CI / lock (push) Successful in 54s
CI / lint (push) Failing after 45s
CI / test (push) Failing after 1m9s
CI / tests-done (push) Failing after 5s
CI / build (push) Successful in 1m34s
CI / publish (push) Has been skipped
to 4253232202
Some checks failed
CI / lock (push) Successful in 48s
CI / test (push) Failing after 1m13s
CI / lint (push) Successful in 1m24s
CI / build (push) Successful in 1m36s
CI / tests-done (push) Failing after 11s
CI / publish (push) Has been skipped
2026-10-15 22:14:48 +02:00
Compare
Author
Member

@hansegucker wrote in #73 (comment):

@magicfelix What's the status of this?

I resolved the conflicts with the current master now. In a quick test, the profile CRUD list was broken. Creating a new profile was not possible, the dialog did not open and the following errors appeared in the browser console:

Uncaught (in promise) TypeError: can't access property "endsWith", schema.type is undefined
    generateField useObjectForm.js:23
    generateFields useObjectForm.js:55
    generateFields useObjectForm.js:51
    useObjectForm useObjectForm.js:169
    setup ObjectForm.vue:18
    callWithErrorHandling runtime-core.esm-bundler.js:199
    setupStatefulComponent runtime-core.esm-bundler.js:8320
useObjectForm.js:23:14
Uncaught (in promise) TypeError: can't access property "classList", data.root.value is undefined
    blockScrollStrategy scrollStrategies.js:51
    useScrollStrategies scrollStrategies.js:30
    run reactivity.esm-bundler.js:92
    useScrollStrategies scrollStrategies.js:26
    callWithErrorHandling runtime-core.esm-bundler.js:199
    callWithAsyncErrorHandling runtime-core.esm-bundler.js:206
    call runtime-core.esm-bundler.js:893
@hansegucker wrote in https://code.teckids.org/aleksis/AlekSIS-App-Paeda/pulls/73#issuecomment-120009: > @magicfelix What's the status of this? I resolved the conflicts with the current master now. In a quick test, the profile CRUD list was broken. Creating a new profile was not possible, the dialog did not open and the following errors appeared in the browser console: ``` Uncaught (in promise) TypeError: can't access property "endsWith", schema.type is undefined generateField useObjectForm.js:23 generateFields useObjectForm.js:55 generateFields useObjectForm.js:51 useObjectForm useObjectForm.js:169 setup ObjectForm.vue:18 callWithErrorHandling runtime-core.esm-bundler.js:199 setupStatefulComponent runtime-core.esm-bundler.js:8320 useObjectForm.js:23:14 Uncaught (in promise) TypeError: can't access property "classList", data.root.value is undefined blockScrollStrategy scrollStrategies.js:51 useScrollStrategies scrollStrategies.js:30 run reactivity.esm-bundler.js:92 useScrollStrategies scrollStrategies.js:26 callWithErrorHandling runtime-core.esm-bundler.js:199 callWithAsyncErrorHandling runtime-core.esm-bundler.js:206 call runtime-core.esm-bundler.js:893 ```
Some checks failed
CI / lock (push) Successful in 48s
CI / test (push) Failing after 1m13s
CI / lint (push) Successful in 1m24s
CI / build (push) Successful in 1m36s
CI / tests-done (push) Failing after 11s
Required
Details
CI / publish (push) Has been skipped
This pull request has changes conflicting with the target branch.
  • aleksis/apps/paeda/frontend/index.js
  • aleksis/apps/paeda/models.py
  • aleksis/apps/paeda/rules.py
  • aleksis/apps/paeda/schema/__init__.py
  • aleksis/apps/paeda/util/predicates.py
View command line instructions

Manual merge helper

Use this merge commit message when completing the merge manually.

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin 15-rename-categories-to-skills-and-introduce-skillset:15-rename-categories-to-skills-and-introduce-skillset
git switch 15-rename-categories-to-skills-and-introduce-skillset
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
aleksis/AlekSIS-App-Paeda!73
No description provided.