Skip to content

Expanded beslistabel (decision table) section of user manual includin…#44

Open
PricklyMush wants to merge 5 commits into
developfrom
feature/beslistabel-alef-manual
Open

Expanded beslistabel (decision table) section of user manual includin…#44
PricklyMush wants to merge 5 commits into
developfrom
feature/beslistabel-alef-manual

Conversation

@PricklyMush

Copy link
Copy Markdown

…g screenshots

@PricklyMush PricklyMush requested review from a team, sognefjellet and um-ami July 6, 2026 11:25
@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown

Test Results

3 864 tests  ±0   3 864 ✅ ±0   10m 6s ⏱️ +3s
  158 suites ±0       0 💤 ±0 
  158 files   ±0       0 ❌ ±0 

Results for commit 4a1d1fd. ± Comparison against base commit 33eefdf.

♻️ This comment has been updated with latest results.

@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown

ALEF distributions (artifacts are deleted in 2 days):

@sognefjellet sognefjellet left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks for the "Beslistabel" documentation. I did quick check for Markdown layout. Can you please correct these findings?

Comment thread docs/regels/Beslistabel.md
Comment thread docs/regels/Beslistabel.md Outdated
Comment thread docs/regels/Beslistabel.md Outdated
Comment thread docs/regels/Beslistabel.md Outdated
Comment thread docs/regels/Beslistabel.md
@sognefjellet sognefjellet self-assigned this Jul 6, 2026
Kleine aanvulling dat gelijkstellingen en kenmerktoekenningen kunnen worden gebruikt.
Comment thread docs/regels/Beslistabel.md
Comment thread docs/regels/Beslistabel.md

@um-ami um-ami 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.

Heel netjes en duidelijk. Een aantal kleine zaken graag nog aanpassen. Zie commentaar.

@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown

ALEF distributions (artifacts are deleted in 2 days):

@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown

ALEF distributions (artifacts are deleted in 2 days):

…acOS keyboard shortcuts, added missing text, added link to equivalente regels (equivalent rules) paragraph of section
@github-actions

github-actions Bot commented Jul 7, 2026

Copy link
Copy Markdown

ALEF distributions (artifacts are deleted in 2 days):

@github-actions

github-actions Bot commented Jul 7, 2026

Copy link
Copy Markdown

ALEF distributions (artifacts are deleted in 2 days):

@PricklyMush

PricklyMush commented Jul 7, 2026

Copy link
Copy Markdown
Author

@um-ami @sognefjellet All requested changes of the PR should be resolved

@um-ami um-ami 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.

@sognefjellet sognefjellet left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Great work, there is one problem with the rendering (see comment). You can view this when you download ALEF that is commented by the GitHub-bot for this PR and open the documentation. Can you also squash your commits to one? If you need help with that, please let me know.

1 of meer **Conclusiekolommen**, waarin de toe te kennen waarden staan,
1 of meer **Conditiekolommen**, waarin de voorwaarden staan waaraan voldaan moet worden en
**Rijen**, die overeenkomen met een gelijkstellingsregel (equivalente regels worden getoond in de Inspector).
* 1 of meer **Conclusiekolommen**, waarin de toe te kennen waarden staan (de eerste kolom in onderstaande afbeelding),

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This list doesn't work in the Markdown rendering engine that is used to render the docs that are added to ALEF. Add a new line after the bevat:

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.

3 participants