Skip to content

Using ISA create form for admin and non admin page - #54

Closed
burkkyy wants to merge 10 commits into
ytgov:mainfrom
icefoganalytics:tk-63/creating-agreement-from-admin-portal-doesnt-match-sharing-agreement-isa-screen
Closed

burkkyy wants to merge 10 commits into
ytgov:mainfrom
icefoganalytics:tk-63/creating-agreement-from-admin-portal-doesnt-match-sharing-agreement-isa-screen

Conversation

@burkkyy

@burkkyy burkkyy commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

Fixes https://yg-hpw.atlassian.net/browse/TK-63

Testing Instructions

  1. Run the test suite via dev test (or dev test_api)
  2. Boot the app via dev up
  3. Log in to the app at http://localhost:8080
  4. Go to Administration.
  5. Click on Agreement → Create ISA.
  6. Observe the ISA creation screen.
  7. Go to Sharing Agreement (Menu).
  8. Create ISA from this menu.
  9. Compare the two ISA creation screens.

klondikemarlen and others added 10 commits April 17, 2026 12:22
Use negative PID to kill entire process group instead of just parent
process. Ensures bridge can be properly shut down when spawned with
pgroup: true.
…tes/information-sharing-agreements/confidentiality-acknowledgement-template.docx` was not being populated by `api/src/serializers/information-sharing-agreements/generate-confidentiality-acknowledgement/create-serializer.ts`
…defined-at-access-from-isa

Populate key `access_level_department_restriction` in CA template
…er-takes-you-to-internal-user-edit-screen

External User Edit form for external users
@burkkyy burkkyy changed the title Tk 63/creating agreement from admin portal doesnt match sharing agreement isa screen Using ISA create form for admin and non admin page Jun 5, 2026
@burkkyy burkkyy closed this Jun 5, 2026
datajohnson pushed a commit that referenced this pull request Aug 28, 2026
…layout-for-sharing-agreement-users-groups

TK-64: Improve Breadcrumb layout for Sharing Agreement, Users, Groups
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