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

Allow CKB users to cross reference EB documents #605

Open
wants to merge 2 commits into
base: feature/submission-cross-referencing
Choose a base branch
from

Conversation

Windvis
Copy link
Contributor

@Windvis Windvis commented Oct 16, 2024

Builds on #575 and allows CKB users to cross reference documents submitted by EBs they govern.

A new field is displayed in the relevant forms that makes it possible to search for documents in a similar fashion as the other cross reference field.

Related tickets
DL-6202, DL-6203, DL-6206, DL-6207

Test instructions

Basic happy flow

  1. log in as an EB user that has a CKB. (Grobbendonk has both a CKB and 2 EB that belong to it)
  2. create a new submission in toezicht with one of the following types and submit it:
  • Jaarrekeningen
  • Budget(wijziging) - Indiening bij centraal bestuur of representatief orgaan
  • Meerjarenplan(aanpassing)
  1. log out and log back in as the corresponding CKB
  2. create a new submission in toezicht with the type that corresponds to the submission you made before:
  • Jaarrekeningen van besturen van de eredienst
  • Budgetten(wijzigingen) - Indiening bij representatief orgaan
  • Budgetten(wijzigingen) - Indiening bij toezichthoudende gemeente of provincie
  • Meerjarenplannen(wijzigingen) van de besturen van de eredienst
  1. You should see a "Gerelateerde documenten" field
  2. click the "Voeg documenten toe" button
  3. select the EB that you used to create the submissions
  4. select the submission and add it to the list (you should only see documents with the correct type, not all documents submitted by the EB)
  5. save and reload the form
  6. the table should load as expected again

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This file doesn't include the changes from this PR, since it wasn't merged when I generated this form. Before we merge this we should regenerate the form and update tho form.ttl so that those changes aren't lost.

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.

2 participants