Skip to content

#281, 313 - Conditional Logic Modal Implementation #332

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

Merged
merged 36 commits into from
Aug 12, 2025

Conversation

SaqAsh
Copy link
Contributor

@SaqAsh SaqAsh commented Jul 29, 2025

Summary

Implemented the conditional logic modal rendering using previously implemented helper functions to recursively render the conditional blocks

Issues

Readiness Checklist

  • Self Review
    • I have performed a self review of code
    • I have run the application locally and manually tested the feature
    • I have checked all updates to correct typos and misspellings
  • Formatting
    • Code follows the project style guide
    • Autmated code formatters (ie. Prettier) have been run
  • Local Testing
    • Successfully built all packages locally
    • Successfully ran all test suites, all unit and integration tests pass
  • Updated Tests
    • Unit and integration tests have been added that describe the bug that was fixed or the features that were added
  • Documentation
    • All new environment variables added to .env.schema file and documented in the README
    • All changes to server HTTP endpoints have open-api documentation
    • All new functions exported from their module have TSDoc comment documentation

SaqAsh added 2 commits July 29, 2025 11:10
commit 138d7873836ca8259a08a2d84288ae0fc26792db
Author: Saqib Ashraf <[email protected]>
Date:   Tue Jul 29 10:58:39 2025 -0400

    Squashed commit of the following:

    commit 73096de907a2eac3314a365505280bd31168e899
    Author: Saqib Ashraf <[email protected]>
    Date:   Tue Jul 29 10:57:17 2025 -0400

        chore: css cleanup for tables and all previous components, and add TS doc where needed (#330)

        * merge in the demo branch

        * remove the pages to reduce diff

        * add ts doc for the accordion component as well as add a TODO for the implementation of the actual dictionary download individual buttons

        * remove the dictionary download button dependency and add a todo instead

        * accordion cleanup

        * fix backwards logic

        * fix the bugs inside of the version switcher due to the new types

        * border changes

        * remove all hard coded colors

        * remove uneccssary comments

        * change the dictionary header to match the figma and fix fonting

        * dictionary

        * POTENTIAL CONFIG CHANGES, PLEASE CONFIRM TO FIX BUILD

        * Squashed commit of the following:

        commit 109d166877befa3b710106de5415eef56d79da89
        Author: Saqib Ashraf <[email protected]>
        Date:   Wed Jul 23 10:45:06 2025 -0400

            variable clean up

        commit 4f22163d6c2ed261ae840f2fa8d96eddd241f08f
        Author: Saqib Ashraf <[email protected]>
        Date:   Wed Jul 23 10:39:41 2025 -0400

            remove import of schema

        commit b57504e67fd9c7f91a4c337594b9d7b900baf334
        Author: Saqib Ashraf <[email protected]>
        Date:   Wed Jul 23 10:38:51 2025 -0400

            changed some of the types and added ts doc

        commit 1d8f597e037940fc41979f619c3dd49029619944
        Merge: bfb6826 7509dae
        Author: Saqib Ashraf <[email protected]>
        Date:   Tue Jul 22 13:43:02 2025 -0400

            Merge remote-tracking branch 'origin/feature/allowed-values-for-entity-relationships' into feature/allowed-values-for-entity-relationships

        commit bfb6826c99c23abb054d41ec705a0549dc91bca2
        Author: Saqib Ashraf <[email protected]>
        Date:   Tue Jul 22 13:42:46 2025 -0400

            add keys

        commit 7509daeabf6720a98fadf8782d0e92e43f2dc2d7
        Merge: 922afce 5e3d846
        Author: Saqib Ashraf <[email protected]>
        Date:   Tue Jul 22 13:42:30 2025 -0400

            Merge branch 'main' into feature/allowed-values-for-entity-relationships

        commit 922afce6ff6051313c03a9ca55ea821339c7cebb
        Author: Saqib Ashraf <[email protected]>
        Date:   Tue Jul 22 13:37:34 2025 -0400

            handle the cases for range

        commit 0bce83a54d987609fe4ca5bcffb458a61ac7fc04
        Merge: 90994a6 d8cd15b
        Author: Saqib Ashraf <[email protected]>
        Date:   Fri Jul 18 11:16:52 2025 -0400

            Merge remote-tracking branch 'origin/feature/allowed-values-for-entity-relationships' into feature/allowed-values-for-entity-relationships

        commit 90994a6758c4a63adef2b6052ef4daaf8079aa1f
        Author: Saqib Ashraf <[email protected]>
        Date:   Fri Jul 18 11:16:45 2025 -0400

            fix uniqueness case

        commit d8cd15b512a6fd902fa31b78074783f898d2204a
        Merge: 854ccd4 b41a4d6
        Author: Saqib Ashraf <[email protected]>
        Date:   Fri Jul 18 10:43:25 2025 -0400

            Merge branch 'main' into feature/allowed-values-for-entity-relationships

        commit 854ccd48dd2fc5e251ae2610fbd7900b48b2c263
        Author: Saqib Ashraf <[email protected]>
        Date:   Fri Jul 18 09:59:43 2025 -0400

            ui cleanup and logic cleanup

        commit e9612c72170a5ee929ed38af6527f58ed8699eb2
        Author: Saqib Ashraf <[email protected]>
        Date:   Thu Jul 17 16:01:51 2025 -0400

            remove the fragment

        commit cbd45a3eaef1ab325145e0b6e7b3b1216f895c5c
        Author: Saqib Ashraf <[email protected]>
        Date:   Thu Jul 17 15:56:24 2025 -0400

            add an and to the list

        commit 7f42c399a534230596495c8280b4296b78f770f8
        Author: Saqib Ashraf <[email protected]>
        Date:   Thu Jul 17 15:50:15 2025 -0400

            clean up the workout.

        commit b8160bb72998607580344bce0b3af2566fa18a73
        Merge: 6c2214c 7e5dd60
        Author: Saqib Ashraf <[email protected]>
        Date:   Thu Jul 17 15:45:24 2025 -0400

            Merge remote-tracking branch 'origin/feature/allowed-values-for-entity-relationships' into feature/allowed-values-for-entity-relationships

        commit 6c2214c543701faefa370c8f10884f15bd8d85da
        Author: Saqib Ashraf <[email protected]>
        Date:   Thu Jul 17 15:45:10 2025 -0400

            some code cleanup after some suggestions as well as fixing the containers

        commit 7e5dd60378b2afb6edcf9dd7fa4695850a3aa3f7
        Merge: 824d998 6d9f613
        Author: Saqib Ashraf <[email protected]>
        Date:   Thu Jul 17 12:03:15 2025 -0400

            Merge branch 'main' into feature/allowed-values-for-entity-relationships

        commit 824d99836a446772a4c03b10fb1ff4c0e27bfcbd
        Author: Saqib Ashraf <[email protected]>
        Date:   Thu Jul 17 11:59:54 2025 -0400

            fix with bolding

        commit c543ea36155f726075af30f66e428536ce7e1869
        Author: Saqib Ashraf <[email protected]>
        Date:   Thu Jul 17 11:57:49 2025 -0400

            field name fix

        commit 8baf36fc3fb8f26555827c1066b428964f61f5b0
        Author: Saqib Ashraf <[email protected]>
        Date:   Thu Jul 17 11:27:39 2025 -0400

            allow multiple relevant foreign keys

        commit c92c0876496516f52e49f96e403e9399bdf05d13
        Author: Saqib Ashraf <[email protected]>
        Date:   Thu Jul 17 09:54:59 2025 -0400

            fix some of the read more text behaviour

        commit 83cdeb350df56f0ae9c7d6af50987cd9740506f2
        Author: Saqib Ashraf <[email protected]>
        Date:   Thu Jul 17 09:01:53 2025 -0400

            cleanup

        commit c24db522a30c52c40efb7e2681e5d181b1fc190e
        Author: Saqib Ashraf <[email protected]>
        Date:   Wed Jul 16 16:41:19 2025 -0400

            hacked together fix for truncation issues.

        commit 52dfcf1df8345fe5c10c6b9266c48a2b7259e33e
        Author: Saqib Ashraf <[email protected]>
        Date:   Wed Jul 16 16:10:39 2025 -0400

            max lines added.

        commit f33dce5fc3e6ba0e8f3f95dff33a76294c5e8bfa
        Author: Saqib Ashraf <[email protected]>
        Date:   Wed Jul 16 16:10:08 2025 -0400

            revert back to old table row.

        commit b084310023998432ebafe24ea9b2e15e075d631a
        Author: Saqib Ashraf <[email protected]>
        Date:   Wed Jul 16 16:03:06 2025 -0400

            add complex stories as requested.

        commit 48c0ef0dbd24464b1e91979ea37e8c088a1fae38
        Author: Saqib Ashraf <[email protected]>
        Date:   Wed Jul 16 15:06:27 2025 -0400

            fix the issue for duplicate read more values

        commit 29295fec85af7cef586bc7d60da384f9b7ffd198
        Author: Saqib Ashraf <[email protected]>
        Date:   Wed Jul 16 14:07:41 2025 -0400

            rename

        commit 5cc11aa0c978cad5f53af966dded66ddd88ed460
        Author: Saqib Ashraf <[email protected]>
        Date:   Wed Jul 16 14:02:51 2025 -0400

            fix naming

        commit 02a988e00a3419f77e28858631966cf52064e177
        Author: Saqib Ashraf <[email protected]>
        Date:   Wed Jul 16 14:01:36 2025 -0400

            add back unique after deletion

        commit befbc7ede8f8e994c1bbae6b5fdeb68e6ca5b4ae
        Author: Saqib Ashraf <[email protected]>
        Date:   Wed Jul 16 12:45:58 2025 -0400

            delete unwanted file

        commit 5d9cd11b6c99c7bc5f9d67dd0ec2e51dbfa50a21
        Author: Saqib Ashraf <[email protected]>
        Date:   Wed Jul 16 12:37:44 2025 -0400

            fix the errors when merging branch over

        commit f9ed5dae22f9f0f5348dac61aa6d90d497d727f7
        Merge: 73c77a3 08a114d
        Author: Saqib Ashraf <[email protected]>
        Date:   Wed Jul 16 12:28:11 2025 -0400

            Merge branch 'main' into feature/allowed-values-for-entity-relationships

        commit 73c77a37e4aa520a548d4cc8b6e6f26457a8a448
        Author: Saqib Ashraf <[email protected]>
        Date:   Wed Jul 16 12:26:19 2025 -0400

            hopefully fix issue

        commit 050ac5ea1e707a264bb126c3305de5cb8b11ee7d
        Author: Saqib Ashraf <[email protected]>
        Date:   Wed Jul 16 11:19:23 2025 -0400

            rendering all cases without individually checking all cases inside of the render file

        commit 2e183edd9c15f63485bcc0b8dbc99edbc331f90b
        Author: Saqib Ashraf <[email protected]>
        Date:   Tue Jul 15 16:49:56 2025 -0400

            hacked together the multi key version

        commit baf78b7cc2e8c4709dde72e2b029d5e3b9cb7a45
        Author: Saqib Ashraf <[email protected]>
        Date:   Tue Jul 15 15:08:37 2025 -0400

            only working in the case of one foreignKey however wil expand in next commit

        commit 4ba5a1e12ddf573cd3342f7fa7405f7f93d7f726
        Author: Saqib Ashraf <[email protected]>
        Date:   Tue Jul 15 14:35:58 2025 -0400

            feat: implement all requirements from the compute restrictions table except for handling multiple, foreign Keys.

        commit e987e8e57a0bcf70ce1f0509a0904564afb7732d
        Author: Saqib Ashraf <[email protected]>
        Date:   Tue Jul 15 11:23:09 2025 -0400

            add cases

        commit 7cb5408c82e9018a083ec5acac2cd147b95f9006
        Author: Saqib Ashraf <[email protected]>
        Date:   Mon Jul 14 16:42:41 2025 -0400

            feat: add ContentType to distinguish between code and string

        commit c71ad7ddc1839e4e4d98e6426956f81bc0b0d2ad
        Author: Saqib Ashraf <[email protected]>
        Date:   Mon Jul 14 12:03:18 2025 -0400

            remove the pill rendering from the fields column

        commit c8a97dc7ca690db53a26a1766863d94e1ca7d8b8
        Author: Saqib Ashraf <[email protected]>
        Date:   Mon Jul 14 12:01:03 2025 -0400

            fix: remove unique pill away from the attribute column. Need to add it to the wording

        * more css changes

        * Squashed commit of the following:

        commit 18c35d4c779c6374af18bcec118e9186ed2000f9
        Author: Saqib Ashraf <[email protected]>
        Date:   Thu Jul 24 13:21:20 2025 -0400

            * fix: remove unique pill away from the attribute column. Need to add it to the wording

            * remove the pill rendering from the fields column

            * feat: add ContentType to distinguish between code and string

            * add cases

            * feat: implement all requirements from the compute restrictions table except for handling multiple, foreign Keys.

            * only working in the case of one foreignKey however wil expand in next commit

            * hacked together the multi key version

            * rendering all cases without individually checking all cases inside of the render file

            * hopefully fix issue

            * fix the errors when merging branch over

            * delete unwanted file

            * add back unique after deletion

            * fix naming

            * rename

            * fix the issue for duplicate read more values

            * add complex stories as requested.

            * revert back to old table row.

            * max lines added.

            * hacked together fix for truncation issues.

            * cleanup

            * fix some of the read more text behaviour

            * allow multiple relevant foreign keys

            * field name fix

            * fix with bolding

            * some code cleanup after some suggestions as well as fixing the containers

            * clean up the workout.

            * add an and to the list

            * remove the fragment

            * ui cleanup and logic cleanup

            * fix uniqueness case

            * handle the cases for range

            * add keys

            * changed some of the types and added ts doc

            * remove import of schema

            * variable clean up

            * accidentally forgot to remove this function that is not being used

        commit 1a345657cf2b49b0ca41c8aaac559ad6a2132feb
        Author: Saqib Ashraf <[email protected]>
        Date:   Thu Jul 24 11:52:39 2025 -0400

            add doccumentation for testers, such as the BioInfoTeam (#327)

        * fix build issues

        * Squashed commit of the following:

        commit 18c35d4c779c6374af18bcec118e9186ed2000f9
        Author: Saqib Ashraf <[email protected]>
        Date:   Thu Jul 24 13:21:20 2025 -0400

            * fix: remove unique pill away from the attribute column. Need to add it to the wording

            * remove the pill rendering from the fields column

            * feat: add ContentType to distinguish between code and string

            * add cases

            * feat: implement all requirements from the compute restrictions table except for handling multiple, foreign Keys.

            * only working in the case of one foreignKey however wil expand in next commit

            * hacked together the multi key version

            * rendering all cases without individually checking all cases inside of the render file

            * hopefully fix issue

            * fix the errors when merging branch over

            * delete unwanted file

            * add back unique after deletion

            * fix naming

            * rename

            * fix the issue for duplicate read more values

            * add complex stories as requested.

            * revert back to old table row.

            * max lines added.

            * hacked together fix for truncation issues.

            * cleanup

            * fix some of the read more text behaviour

            * allow multiple relevant foreign keys

            * field name fix

            * fix with bolding

            * some code cleanup after some suggestions as well as fixing the containers

            * clean up the workout.

            * add an and to the list

            * remove the fragment

            * ui cleanup and logic cleanup

            * fix uniqueness case

            * handle the cases for range

            * add keys

            * changed some of the types and added ts doc

            * remove import of schema

            * variable clean up

            * accidentally forgot to remove this function that is not being used

        commit 1a345657cf2b49b0ca41c8aaac559ad6a2132feb
        Author: Saqib Ashraf <[email protected]>
        Date:   Thu Jul 24 11:52:39 2025 -0400

            add doccumentation for testers, such as the BioInfoTeam (#327)

        * Squashed commit of the following:

        commit 18c35d4c779c6374af18bcec118e9186ed2000f9
        Author: Saqib Ashraf <[email protected]>
        Date:   Thu Jul 24 13:21:20 2025 -0400

            * fix: remove unique pill away from the attribute column. Need to add it to the wording

            * remove the pill rendering from the fields column

            * feat: add ContentType to distinguish between code and string

            * add cases

            * feat: implement all requirements from the compute restrictions table except for handling multiple, foreign Keys.

            * only working in the case of one foreignKey however wil expand in next commit

            * hacked together the multi key version

            * rendering all cases without individually checking all cases inside of the render file

            * hopefully fix issue

            * fix the errors when merging branch over

            * delete unwanted file

            * add back unique after deletion

            * fix naming

            * rename

            * fix the issue for duplicate read more values

            * add complex stories as requested.

            * revert back to old table row.

            * max lines added.

            * hacked together fix for truncation issues.

            * cleanup

            * fix some of the read more text behaviour

            * allow multiple relevant foreign keys

            * field name fix

            * fix with bolding

            * some code cleanup after some suggestions as well as fixing the containers

            * clean up the workout.

            * add an and to the list

            * remove the fragment

            * ui cleanup and logic cleanup

            * fix uniqueness case

            * handle the cases for range

            * add keys

            * changed some of the types and added ts doc

            * remove import of schema

            * variable clean up

            * accidentally forgot to remove this function that is not being used

        commit 1a345657cf2b49b0ca41c8aaac559ad6a2132feb
        Author: Saqib Ashraf <[email protected]>
        Date:   Thu Jul 24 11:52:39 2025 -0400

            add doccumentation for testers, such as the BioInfoTeam (#327)

        * Squashed commit of the following:

        commit 18c35d4c779c6374af18bcec118e9186ed2000f9
        Author: Saqib Ashraf <[email protected]>
        Date:   Thu Jul 24 13:21:20 2025 -0400

            * fix: remove unique pill away from the attribute column. Need to add it to the wording

            * remove the pill rendering from the fields column

            * feat: add ContentType to distinguish between code and string

            * add cases

            * feat: implement all requirements from the compute restrictions table except for handling multiple, foreign Keys.

            * only working in the case of one foreignKey however wil expand in next commit

            * hacked together the multi key version

            * rendering all cases without individually checking all cases inside of the render file

            * hopefully fix issue

            * fix the errors when merging branch over

            * delete unwanted file

            * add back unique after deletion

            * fix naming

            * rename

            * fix the issue for duplicate read more values

            * add complex stories as requested.

            * revert back to old table row.

            * max lines added.

            * hacked together fix for truncation issues.

            * cleanup

            * fix some of the read more text behaviour

            * allow multiple relevant foreign keys

            * field name fix

            * fix with bolding

            * some code cleanup after some suggestions as well as fixing the containers

            * clean up the workout.

            * add an and to the list

            * remove the fragment

            * ui cleanup and logic cleanup

            * fix uniqueness case

            * handle the cases for range

            * add keys

            * changed some of the types and added ts doc

            * remove import of schema

            * variable clean up

            * accidentally forgot to remove this function that is not being used

        commit 1a345657cf2b49b0ca41c8aaac559ad6a2132feb
        Author: Saqib Ashraf <[email protected]>
        Date:   Thu Jul 24 11:52:39 2025 -0400

            add doccumentation for testers, such as the BioInfoTeam (#327)

        * fix for broken scroll

        * fix the bolding

        * get rid of the functional use of the serialized styles

        * add TSDoc to componets

        * add licensing

        * add licencing

        * READ MORE COMPONENT ACTING FUNKY

        * added threshold for the readmore text since the line height of the text is distrupting the truncation logic

        * fix readmore again

        * typing the theme for better use and add the licensing

        * Clean up imports as well as licensing and useClipboard inside of the utils

        * scroll shadow fix

        * add TODO for dropdown item styling

        * change labeling to make more sense

        * compose the typography, reducing lines a lot, as well as move useClipboard over

        * add back the icon that was gone from the list filter dropdown

        * clean up the dropdown menu, according to ux requirements

        * unset and padding removal

        * use the typography for the listItem

        * get rid of the use of both css as well as styles and make it more flexible

        * add pill stories and compose the css

        * base styles no longer a fn

        * fixing ordering for imports inside of stories

commit aaec6dda77871b9492f11156942bf811ef750730
Merge: ffcd6d5 1aecbf6
Author: Saqib Ashraf <[email protected]>
Date:   Tue Jul 29 10:44:54 2025 -0400

    Merge branch 'chore/css-fixes' into feature/if-then-else-quote-blocks

commit 1aecbf6edaa6dd4d5ba4e564d76690453c6a5e6f
Author: Saqib Ashraf <[email protected]>
Date:   Tue Jul 29 10:17:51 2025 -0400

    fixing ordering for imports inside of stories

commit 5f1f20164600cf7e7ca7ae60f11b121de6538562
Author: Saqib Ashraf <[email protected]>
Date:   Tue Jul 29 10:17:09 2025 -0400

    base styles no longer a fn

commit 3718195b67faaaabcd18031c7388be94459401aa
Author: Saqib Ashraf <[email protected]>
Date:   Tue Jul 29 10:13:35 2025 -0400

    add pill stories and compose the css

commit 18ea2564c6fd27c662821256630b793b7f947c78
Author: Saqib Ashraf <[email protected]>
Date:   Tue Jul 29 09:55:35 2025 -0400

    get rid of the use of both css as well as styles and make it more flexible

commit 6fba2b580be3a4227a7d93f104517430a96febc6
Author: Saqib Ashraf <[email protected]>
Date:   Mon Jul 28 22:08:53 2025 -0400

    use the typography for the listItem

commit ea53c6d77806da6dd7553943a19dc5ae5b5ff3ab
Merge: c6463d3 f4217eb
Author: Saqib Ashraf <[email protected]>
Date:   Mon Jul 28 21:41:02 2025 -0400

    Merge branch 'main' into chore/css-fixes

commit c6463d3783dd190f38d5693f88bbccd4ed2170b4
Author: Saqib Ashraf <[email protected]>
Date:   Mon Jul 28 14:36:48 2025 -0400

    unset and padding removal

commit af0d3edc9ba284e49a46e8360e86894b6506f167
Author: Saqib Ashraf <[email protected]>
Date:   Mon Jul 28 14:09:18 2025 -0400

    clean up the dropdown menu, according to ux requirements

commit ce00d35881cc6235a2f9b4d53a8646844bc157aa
Author: Saqib Ashraf <[email protected]>
Date:   Mon Jul 28 13:54:16 2025 -0400

    add back the icon that was gone from the list filter dropdown

commit ffcd6d5bc21a978fdee67f729db94140a6286bf0
Author: Saqib Ashraf <[email protected]>
Date:   Mon Jul 28 13:51:46 2025 -0400

    story changes

commit 18755005eb7c3527e6fc7ab8ec2e8f3695a8645d
Author: Saqib Ashraf <[email protected]>
Date:   Mon Jul 28 13:32:39 2025 -0400

    get rid of else statement, better to do it without it

commit 0f08a1fffc0d5a7e80bc2a0668f4d8709f9bc674
Author: Saqib Ashraf <[email protected]>
Date:   Mon Jul 28 13:18:14 2025 -0400

    add ts doc and handle condition where we have both conditional and non conditionals

commit c25d8c73c233dfbb4a2394416df40385e15063f7
Author: Saqib Ashraf <[email protected]>
Date:   Mon Jul 28 11:05:34 2025 -0400

    container fix, need to still simplify the logic

commit 443c7764365062351ef2e6d005a11a69fe54eaf2
Author: Saqib Ashraf <[email protected]>
Date:   Mon Jul 28 10:25:14 2025 -0400

    compose the typography, reducing lines a lot, as well as move useClipboard over

commit 329daba541d1262f6a8a32ab47f1037d3b1a84f9
Author: Saqib Ashraf <[email protected]>
Date:   Mon Jul 28 09:26:01 2025 -0400

    remove un-needed import

commit ae598dd6e62d7c4dbfdd8edf202cf7857bfcd4bc
Author: Saqib Ashraf <[email protected]>
Date:   Fri Jul 25 17:09:59 2025 -0400

    conditional logic should be almost done

commit 23003f47e79af8c913c2709687284848ed0f4a59
Author: Saqib Ashraf <[email protected]>
Date:   Fri Jul 25 14:52:48 2025 -0400

    if statements looking good, however there is some confusion about the then statement that needs to be looked at next

commit dfdcc6159e9e4f089322da521b6f3a8116afa206
Author: Saqib Ashraf <[email protected]>
Date:   Fri Jul 25 12:53:05 2025 -0400

    more error

commit 550714ea202a6feb9ff42dbe34414dfe3d14d28b
Author: Saqib Ashraf <[email protected]>
Date:   Fri Jul 25 12:49:04 2025 -0400

    fix the issues from the merging, very annoying

commit 0f132fc871a5cafbb97b591ed8a5ade10ebe21fd
Author: Saqib Ashraf <[email protected]>
Date:   Fri Jul 25 12:43:31 2025 -0400

    compute allowed values

commit 7775855ca0634737976ffb78a801a248ec947935
Author: Saqib Ashraf <[email protected]>
Date:   Fri Jul 25 12:43:12 2025 -0400

    Squashed commit of the following:

    commit 74a9e02a702417d582929a27165df3b2b82c0ff8
    Author: Chiso <[email protected]>
    Date:   Thu Jul 24 14:58:14 2025 -0400

        Feat: add display name property to field and Schema  (#331)

        * Add optional displayName field to SchemaFieldBase and Schema

        * Add displayName property to schema definitions and update documentation

        * Add tests for displayName field in SchemaField and Schema

        * ⛏️: remove space in displayName Test

        * cleanup: prettier format for docs alignment

    commit 18c35d4c779c6374af18bcec118e9186ed2000f9
    Author: Saqib Ashraf <[email protected]>
    Date:   Thu Jul 24 13:21:20 2025 -0400

        * fix: remove unique pill away from the attribute column. Need to add it to the wording

        * remove the pill rendering from the fields column

        * feat: add ContentType to distinguish between code and string

        * add cases

        * feat: implement all requirements from the compute restrictions table except for handling multiple, foreign Keys.

        * only working in the case of one foreignKey however wil expand in next commit

        * hacked together the multi key version

        * rendering all cases without individually checking all cases inside of the render file

        * hopefully fix issue

        * fix the errors when merging branch over

        * delete unwanted file

        * add back unique after deletion

        * fix naming

        * rename

        * fix the issue for duplicate read more values

        * add complex stories as requested.

        * revert back to old table row.

        * max lines added.

        * hacked together fix for truncation issues.

        * cleanup

        * fix some of the read more text behaviour

        * allow multiple relevant foreign keys

        * field name fix

        * fix with bolding

        * some code cleanup after some suggestions as well as fixing the containers

        * clean up the workout.

        * add an and to the list

        * remove the fragment

        * ui cleanup and logic cleanup

        * fix uniqueness case

        * handle the cases for range

        * add keys

        * changed some of the types and added ts doc

        * remove import of schema

        * variable clean up

        * accidentally forgot to remove this function that is not being used

    commit 1a345657cf2b49b0ca41c8aaac559ad6a2132feb
    Author: Saqib Ashraf <[email protected]>
    Date:   Thu Jul 24 11:52:39 2025 -0400

        add doccumentation for testers, such as the BioInfoTeam (#327)

commit 300464b592237ed5966168508f7e02d37795b881
Author: Saqib Ashraf <[email protected]>
Date:   Fri Jul 25 12:39:49 2025 -0400

    Squashed commit of the following:

    commit 6817edf107f7eb02916c00762ae031c555eeba14
    Author: Saqib Ashraf <[email protected]>
    Date:   Fri Jul 25 12:10:46 2025 -0400

        change labeling to make more sense

    commit e00bba0b37a1450b8535c28a317561101add7bc6
    Author: Saqib Ashraf <[email protected]>
    Date:   Fri Jul 25 12:05:08 2025 -0400

        add TODO for dropdown item styling

    commit a0c653b929e9f9a8cf352d258bb28f0ca4f7a7a6
    Author: Saqib Ashraf <[email protected]>
    Date:   Fri Jul 25 11:14:49 2025 -0400

        scroll shadow fix

    commit f618df737aec1161b745ca4c16d29cf24f970066
    Author: Saqib Ashraf <[email protected]>
    Date:   Fri Jul 25 10:55:22 2025 -0400

        Clean up imports as well as licensing and useClipboard inside of the utils

    commit 821fa2bb21f09107cf9d95e819c92eebcbfe509e
    Author: Saqib Ashraf <[email protected]>
    Date:   Fri Jul 25 10:14:54 2025 -0400

        typing the theme for better use and add the licensing

    commit 77ab971dc5b7878963e2f4335b2d35152be7e207
    Merge: 4ef04d0 74a9e02
    Author: Saqib Ashraf <[email protected]>
    Date:   Fri Jul 25 09:57:57 2025 -0400

        Merge branch 'main' into chore/css-fixes

    commit 4ef04d00a06d6b112ad1e4053cdc35cc7a8d3f1f
    Author: Saqib Ashraf <[email protected]>
    Date:   Fri Jul 25 09:56:48 2025 -0400

        fix readmore again

    commit 801a2022abf04f8e9e321c5254197c47b7eeab2a
    Author: Saqib Ashraf <[email protected]>
    Date:   Thu Jul 24 16:24:52 2025 -0400

        added threshold for the readmore text since the line height of the text is distrupting the truncation logic

    commit 07c298f8a14c2cf8ac198f536d133823d21fd9bb
    Author: Saqib Ashraf <[email protected]>
    Date:   Thu Jul 24 16:21:30 2025 -0400

        READ MORE COMPONENT ACTING FUNKY

    commit 1a470c23d3751bc6e25e9e099181b78139e74379
    Author: Saqib Ashraf <[email protected]>
    Date:   Thu Jul 24 16:13:42 2025 -0400

        add licencing

    commit 4dfd1c0ef7df24df18590a45d16e7477b1d61a4f
    Author: Saqib Ashraf <[email protected]>
    Date:   Thu Jul 24 16:13:36 2025 -0400

        add licensing

    commit 6e4511c220b8703d02404fb132f9c9cd45854748
    Author: Saqib Ashraf <[email protected]>
    Date:   Thu Jul 24 15:45:39 2025 -0400

        add TSDoc to componets

    commit a7e06d4c0a394a40d76ec65ac003de2cea98c6a9
    Author: Saqib Ashraf <[email protected]>
    Date:   Thu Jul 24 15:02:43 2025 -0400

        get rid of the functional use of the serialized styles

    commit 74a9e02a702417d582929a27165df3b2b82c0ff8
    Author: Chiso <[email protected]>
    Date:   Thu Jul 24 14:58:14 2025 -0400

        Feat: add display name property to field and Schema  (#331)

        * Add optional displayName field to SchemaFieldBase and Schema

        * Add displayName property to schema definitions and update documentation

        * Add tests for displayName field in SchemaField and Schema

        * ⛏️: remove space in displayName Test

        * cleanup: prettier format for docs alignment

    commit f14e1612623f347f3a0bb64cea0ee8e1e61e0ae6
    Author: Saqib Ashraf <[email protected]>
    Date:   Thu Jul 24 14:07:03 2025 -0400

        fix the bolding

    commit f993fc9694fd84beb7f12109e9228a9da8c87f96
    Author: Saqib Ashraf <[email protected]>
    Date:   Thu Jul 24 13:55:53 2025 -0400

        fix for broken scroll

    commit 63e0d8877da52d1241a8028fa012261159cd5316
    Merge: 48b4b9e 18c35d4
    Author: Saqib Ashraf <[email protected]>
    Date:   Thu Jul 24 13:45:47 2025 -0400

        Merge branch 'main' into chore/css-fixes

    commit 48b4b9ead19572f8dddcb378cf1894b817ada914
    Author: Saqib Ashraf <[email protected]>
    Date:   Thu Jul 24 13:45:27 2025 -0400

        Squashed commit of the following:

        commit 18c35d4c779c6374af18bcec118e9186ed2000f9
        Author: Saqib Ashraf <[email protected]>
        Date:   Thu Jul 24 13:21:20 2025 -0400

            * fix: remove unique pill away from the attribute column. Need to add it to the wording

            * remove the pill rendering from the fields column

            * feat: add ContentType to distinguish between code and string

            * add cases

            * feat: implement all requirements from the compute restrictions table except for handling multiple, foreign Keys.

            * only working in the case of one foreignKey however wil expand in next commit

            * hacked together the multi key version

            * rendering all cases without individually checking all cases inside of the render file

            * hopefully fix issue

            * fix the errors when merging branch over

            * delete unwanted file

            * add back unique after deletion

            * fix naming

            * rename

            * fix the issue for duplicate read more values

            * add complex stories as requested.

            * revert back to old table row.

            * max lines added.

            * hacked together fix for truncation issues.

            * cleanup

            * fix some of the read more text behaviour

            * allow multiple relevant foreign keys

            * field name fix

            * fix with bolding

            * some code cleanup after some suggestions as well as fixing the containers

            * clean up the workout.

            * add an and to the list

            * remove the fragment

            * ui cleanup and logic cleanup

            * fix uniqueness case

            * handle the cases for range

            * add keys

            * changed some of the types and added ts doc

            * remove import of schema

            * variable clean up

            * accidentally forgot to remove this function that is not being used

        commit 1a345657cf2b49b0ca41c8aaac559ad6a2132feb
        Author: Saqib Ashraf <[email protected]>
        Date:   Thu Jul 24 11:52:39 2025 -0400

            add doccumentation for testers, such as the BioInfoTeam (#327)

    commit 4f9a1a5f3b95bb709b3d168434f2f2ca2dba4b70
    Author: Saqib Ashraf <[email protected]>
    Date:   Thu Jul 24 13:44:21 2025 -0400

        Squashed commit of the following:

        commit 18c35d4c779c6374af18bcec118e9186ed2000f9
        Author: Saqib Ashraf <[email protected]>
        Date:   Thu Jul 24 13:21:20 2025 -0400

            * fix: remove unique pill away from the attribute column. Need to add it to the wording

            * remove the pill rendering from the fields column

            * feat: add ContentType to distinguish between code and string

            * add cases

            * feat: implement all requirements from the compute restrictions table except for handling multiple, foreign Keys.

            * only working in the case of one foreignKey however wil expand in next commit

            * hacked together the multi key version

            * rendering all cases without individually checking all cases inside of the render file

            * hopefully fix issue

            * fix the errors when merging branch over

            * delete unwanted file

            * add back unique after deletion

            * fix naming

            * rename

            * fix the issue for duplicate read more values

            * add complex stories as requested.

            * revert back to old table row.

            * max lines added.

            * hacked together fix for truncation issues.

            * cleanup

            * fix some of the read more text behaviour

            * allow multiple relevant foreign keys

            * field name fix

            * fix with bolding

            * some code cleanup after some suggestions as well as fixing the containers

            * clean up the workout.

            * add an and to the list

            * remove the fragment

            * ui cleanup and logic cleanup

            * fix uniqueness case

            * handle the cases for range

            * add keys

            * changed some of the types and added ts doc

            * remove import of schema

            * variable clean up

            * accidentally forgot to remove this function that is not being used

        commit 1a345657cf2b49b0ca41c8aaac559ad6a2132feb
        Author: Saqib Ashraf <[email protected]>
        Date:   Thu Jul 24 11:52:39 2025 -0400

            add doccumentation for testers, such as the BioInfoTeam (#327)

    commit 75b5754d2dab338a8e69f6f8cd5314d8f78a4e97
    Author: Saqib Ashraf <[email protected]>
    Date:   Thu Jul 24 13:41:57 2025 -0400

        Squashed commit of the following:

        commit 18c35d4c779c6374af18bcec118e9186ed2000f9
        Author: Saqib Ashraf <[email protected]>
        Date:   Thu Jul 24 13:21:20 2025 -0400

            * fix: remove unique pill away from the attribute column. Need to add it to the wording

            * remove the pill rendering from the fields column

            * feat: add ContentType to distinguish between code and string

            * add cases

            * feat: implement all requirements from the compute restrictions table except for handling multiple, foreign Keys.

            * only working in the case of one foreignKey however wil expand in next commit

            * hacked together the multi key version

            * rendering all cases without individually checking all cases inside of the render file

            * hopefully fix issue

            * fix the errors when merging branch over

            * delete unwanted file

            * add back unique after deletion

            * fix naming

            * rename

            * fix the issue for duplicate read more values

            * add complex stories as requested.

            * revert back to old table row.

            * max lines added.

            * hacked together fix for truncation issues.

            * cleanup

            * fix some of the read more text behaviour

            * allow multiple relevant foreign keys

            * field name fix

            * fix with bolding

            * some code cleanup after some suggestions as well as fixing the containers

            * clean up the workout.

            * add an and to the list

            * remove the fragment

            * ui cleanup and logic cleanup

            * fix uniqueness case

            * handle the cases for range

            * add keys

            * changed some of the types and added ts doc

            * remove import of schema

            * variable clean up

            * accidentally forgot to remove this function that is not being used

        commit 1a345657cf2b49b0ca41c8aaac559ad6a2132feb
        Author: Saqib Ashraf <[email protected]>
        Date:   Thu Jul 24 11:52:39 2025 -0400

            add doccumentation for testers, such as the BioInfoTeam (#327)

    commit d3b698c1d7b3a342c32e78c364dd8870f9006b32
    Author: Saqib Ashraf <[email protected]>
    Date:   Thu Jul 24 13:38:00 2025 -0400

        fix build issues

    commit 5203e910e5f19a5052d6435e26f988e91c44cbe1
    Author: Saqib Ashraf <[email protected]>
    Date:   Thu Jul 24 13:34:34 2025 -0400

        Squashed commit of the following:

        commit 18c35d4c779c6374af18bcec118e9186ed2000f9
        Author: Saqib Ashraf <[email protected]>
        Date:   Thu Jul 24 13:21:20 2025 -0400

            * fix: remove unique pill away from the attribute column. Need to add it to the wording

            * remove the pill rendering from the fields column

            * feat: add ContentType to distinguish between code and string

            * add cases

            * feat: implement all requirements from the compute restrictions table except for handling multiple, foreign Keys.

            * only working in the case of one foreignKey however wil expand in next commit

            * hacked together the multi key version

            * rendering all cases without individually checking all cases inside of the render file

            * hopefully fix issue

            * fix the errors when merging branch over

            * delete unwanted file

            * add back unique after deletion

            * fix naming

            * rename

            * fix the issue for duplicate read more values

            * add complex stories as requested.

            * revert back to old table row.

            * max lines added.

            * hacked together fix for truncation issues.

            * cleanup

            * fix some of the read more text behaviour

            * allow multiple relevant foreign keys

            * field name fix

            * fix with bolding

            * some code cleanup after some suggestions as well as fixing the containers

            * clean up the workout.

            * add an and to the list

            * remove the fragment

            * ui cleanup and logic cleanup

            * fix uniqueness case

            * handle the cases for range

            * add keys

            * changed some of the types and added ts doc

            * remove import of schema

            * variable clean up

            * accidentally forgot to remove this function that is not being used

        commit 1a345657cf2b49b0ca41c8aaac559ad6a2132feb
        Author: Saqib Ashraf <[email protected]>
        Date:   Thu Jul 24 11:52:39 2025 -0400

            add doccumentation for testers, such as the BioInfoTeam (#327)

    commit 18c35d4c779c6374af18bcec118e9186ed2000f9
    Author: Saqib Ashraf <[email protected]>
    Date:   Thu Jul 24 13:21:20 2025 -0400

        * fix: remove unique pill away from the attribute column. Need to add it to the wording

        * remove the pill rendering from the fields column

        * feat: add ContentType to distinguish between code and string

        * add cases

        * feat: implement all requirements from the compute restrictions table except for handling multiple, foreign Keys.

        * only working in the case of one foreignKey however wil expand in next commit

        * hacked together the multi key version

        * rendering all cases without individually checking all cases inside of the render file

        * hopefully fix issue

        * fix the errors when merging branch over

        * delete unwanted file

        * add back unique after deletion

        * fix naming

        * rename

        * fix the issue for duplicate read more values

        * add complex stories as requested.

        * revert back to old table row.

        * max lines added.

        * hacked together fix for truncation issues.

        * cleanup

        * fix some of the read more text behaviour

        * allow multiple relevant foreign keys

        * field name fix

        * fix with bolding

        * some code cleanup after some suggestions as well as fixing the containers

        * clean up the workout.

        * add an and to the list

        * remove the fragment

        * ui cleanup and logic cleanup

        * fix uniqueness case

        * handle the cases for range

        * add keys

        * changed some of the types and added ts doc

        * remove import of schema

        * variable clean up

        * accidentally forgot to remove this function that is not being used

    commit 1a345657cf2b49b0ca41c8aaac559ad6a2132feb
    Author: Saqib Ashraf <[email protected]>
    Date:   Thu Jul 24 11:52:39 2025 -0400

        add doccumentation for testers, such as the BioInfoTeam (#327)

    commit 2b66bdb53c17044f78b59e01291c41c6a2f4ad2c
    Author: Saqib Ashraf <[email protected]>
    Date:   Wed Jul 23 16:57:07 2025 -0400

        more css changes

    commit f4f6adce75392d1dbce8a082c8fd2d2802d7b825
    Author: Saqib Ashraf <[email protected]>
    Date:   Wed Jul 23 16:27:50 2025 -0400

        Squashed commit of the following:

        commit 109d166877befa3b710106de5415eef56d79da89
        Author: Saqib Ashraf <[email protected]>
        Date:   Wed Jul 23 10:45:06 2025 -0400

            variable clean up

        commit 4f22163d6c2ed261ae840f2fa8d96eddd241f08f
        Author: Saqib Ashraf <[email protected]>
        Date:   Wed Jul 23 10:39:41 2025 -0400

            remove import of schema

        commit b57504e67fd9c7f91a4c337594b9d7b900baf334
        Author: Saqib Ashraf <[email protected]>
        Date:   Wed Jul 23 10:38:51 2025 -0400

            changed some of the types and added ts doc

        commit 1d8f597e037940fc41979f619c3dd49029619944
        Merge: bfb6826 7509dae
        Author: Saqib Ashraf <[email protected]>
        Date:   Tue Jul 22 13:43:02 2025 -0400

            Merge remote-tracking branch 'origin/feature/allowed-values-for-entity-relationships' into feature/allowed-values-for-entity-relationships

        commit bfb6826c99c23abb054d41ec705a0549dc91bca2
        Author: Saqib Ashraf <[email protected]>
        Date:   Tue Jul 22 13:42:46 2025 -0400

            add keys

        commit 7509daeabf6720a98fadf8782d0e92e43f2dc2d7
        Merge: 922afce 5e3d846
        Author: Saqib Ashraf <[email protected]>
        Date:   Tue Jul 22 13:42:30 2025 -0400

            Merge branch 'main' into feature/allowed-values-for-entity-relationships

        commit 922afce6ff6051313c03a9ca55ea821339c7cebb
        Author: Saqib Ashraf <[email protected]>
        Date:   Tue Jul 22 13:37:34 2025 -0400

            handle the cases for range

        commit 0bce83a54d987609fe4ca5bcffb458a61ac7fc04
        Merge: 90994a6 d8cd15b
        Author: Saqib Ashraf <[email protected]>
        Date:   Fri Jul 18 11:16:52 2025 -0400

            Merge remote-tracking branch 'origin/feature/allowed-values-for-entity-relationships' into feature/allowed-values-for-entity-relationships

        commit 90994a6758c4a63adef2b6052ef4daaf8079aa1f
        Author: Saqib Ashraf <[email protected]>
        Date:   Fri Jul 18 11:16:45 2025 -0400

            fix uniqueness case

        commit d8cd15b512a6fd902fa31b78074783f898d2204a
        Merge: 854ccd4 b41a4d6
        Author: Saqib Ashraf <[email protected]>
        Date:   Fri Jul 18 10:43:25 2025 -0400

            Merge branch 'main' into feature/allowed-values-for-entity-relationships

        commit 854ccd48dd2fc5e251ae2610fbd7900b48b2c263
        Author: Saqib Ashraf <[email protected]>
        Date:   Fri Jul 18 09:59:43 2025 -0400

            ui cleanup and logic cleanup

        commit e9612c72170a5ee929ed38af6527f58ed8699eb2
        Author: Saqib Ashraf <[email protected]>
        Date:   Thu Jul 17 16:01:51 2025 -0400

            remove the fragment

        commit cbd45a3eaef1ab325145e0b6e7b3b1216f895c5c
        Author: Saqib Ashraf <[email protected]>
        Date:   Thu Jul 17 15:56:24 2025 -0400

            add an and to the list

        commit 7f42c399a534230596495c8280b4296b78f770f8
        Author: Saqib Ashraf <[email protected]>
        Date:   Thu Jul 17 15:50:15 2025 -0400

            clean up the workout.

        commit b8160bb72998607580344bce0b3af2566fa18a73
        Merge: 6c2214c 7e5dd60
        Author: Saqib Ashraf <[email protected]>
        Date:   Thu Jul 17 15:45:24 2025 -0400

            Merge remote-tracking branch 'origin/feature/allowed-values-for-entity-relationships' into feature/allowed-values-for-entity-relationships

        commit 6c2214c543701faefa370c8f10884f15bd8d85da
        Author: Saqib Ashraf <[email protected]>
        Date:   Thu Jul 17 15:45:10 2025 -0400

            some code cleanup after some suggestions as well as fixing the containers

        commit 7e5dd60378b2afb6edcf9dd7fa4695850a3aa3f7
        Merge: 824d998 6d9f613
        Author: Saqib Ashraf <[email protected]>
        Date:   Thu Jul 17 12:03:15 2025 -0400

            Merge branch 'main' into feature/allowed-values-for-entity-relationships

        commit 824d99836a446772a4c03b10fb1ff4c0e27bfcbd
        Author: Saqib Ashraf <[email protected]>
        Date:   Thu Jul 17 11:59:54 2025 -0400

            fix with bolding

        commit c543ea36155f726075af30f66e428536ce7e1869
        Author: Saqib Ashraf <[email protected]>
        Date:   Thu Jul 17 11:57:49 2025 -0400

            field name fix

        commit 8baf36fc3fb8f26555827c1066b428964f61f5b0
        Author: Saqib Ashraf <[email protected]>
        Date:   Thu Jul 17 11:27:39 2025 -0400

            allow multiple relevant foreign keys

        commit c92c0876496516f52e49f96e403e9399bdf05d13
        Author: Saqib Ashraf <[email protected]>
        Date:   Thu Jul 17 09:54:59 2025 -0400

            fix some of the read more text behaviour

        commit 83cdeb350df56f0ae9c7d6af50987cd9740506f2
        Author: Saqib Ashraf <[email protected]>
        Date:   Thu Jul 17 09:01:53 2025 -0400

            cleanup

        commit c24db522a30c52c40efb7e2681e5d181b1fc190e
        Author: Saqib Ashraf <[email protected]>
        Date:   Wed Jul 16 16:41:19 2025 -0400

            hacked together fix for truncation issues.

        commit 52dfcf1df8345fe5c10c6b9266c48a2b7259e33e
        Author: Saqib Ashraf <[email protected]>
        Date:   Wed Jul 16 16:10:39 2025 -0400

            max lines added.

        commit f33dce5fc3e6ba0e8f3f95dff33a76294c5e8bfa
        Author: Saqib Ashraf <[email protected]>
        Date:   Wed Jul 16 16:10:08 2025 -0400

            revert back to old table row.

        commit b084310023998432ebafe24ea9b2e15e075d631a
        Author: Saqib Ashraf <[email protected]>
        Date:   Wed Jul 16 16:03:06 2025 -0400

            add complex stories as requested.

        commit 48c0ef0dbd24464b1e91979ea37e8c088a1fae38
        Author: Saqib Ashraf <[email protected]>
        Date:   Wed Jul 16 15:06:27 2025 -0400

            fix the issue for duplicate read more values

        commit 29295fec85af7cef586bc7d60da384f9b7ffd198
        Author: Saqib Ashraf <[email protected]>
        Date:   Wed Jul 16 14:07:41 2025 -0400

            rename

        commit 5cc11aa0c978cad5f53af966dded66ddd88ed460
        Author: Saqib Ashraf <[email protected]>
        Date:   Wed Jul 16 14:02:51 2025 -0400

            fix naming

        commit 02a988e00a3419f77e28858631966cf52064e177
        Author: Saqib Ashraf <[email protected]>
        Date:   Wed Jul 16 14:01:36 2025 -0400

            add back unique after deletion

        commit befbc7ede8f8e994c1bbae6b5fdeb68e6ca5b4ae
        Author: Saqib Ashraf <[email protected]>
        Date:   Wed Jul 16 12:45:58 2025 -0400

            delete unwanted file

        commit 5d9cd11b6c99c7bc5f9d67dd0ec2e51dbfa50a21
        Author: Saqib Ashraf <[email protected]>
        Date:   Wed Jul 16 12:37:44 2025 -0400

            fix the errors when merging branch over

        commit f9ed5dae22f9f0f5348dac61aa6d90d497d727f7
        Merge: 73c77a3 08a114d
        Author: Saqib Ashraf <[email protected]>
        Date:   Wed Jul 16 12:28:11 2025 -0400

            Merge branch 'main' into feature/allowed-values-for-entity-relationships

        commit 73c77a37e4aa520a548d4cc8b6e6f26457a8a448
        Author: Saqib Ashraf <[email protected]>
        Date:   Wed Jul 16 12:26:19 2025 -0400

            hopefully fix issue

        commit 050ac5ea1e707a264bb126c3305de5cb8b11ee7d
        Author: Saqib Ashraf <[email protected]>
        Date:   Wed Jul 16 11:19:23 2025 -0400

            rendering all cases without individually checking all cases inside of the render file

        commit 2e183edd9c15f63485bcc0b8dbc99edbc331f90b
        Author: Saqib Ashraf <[email protected]>
        Date:   Tue Jul 15 16:49:56 2025 -0400

            hacked together the multi key version

        commit baf78b7cc2e8c4709dde72e2b029d5e3b9cb7a45
        Author: Saqib Ashraf <[email protected]>
        Date:   Tue Jul 15 15:08:37 2025 -0400

            only working in the case of one foreignKey however wil expand in next commit

        commit 4ba5a1e12ddf573cd3342f7fa7405f7f93d7f726
        Author: Saqib Ashraf <[email protected]>
        Date:   Tue Jul 15 14:35:58 2025 -0400

            feat: implement all requirements from the compute restrictions table except for handling multiple, foreign Keys.

        commit e987e8e57a0bcf70ce1f0509a0904564afb7732d
        Author: Saqib Ashraf <[email protected]>
        Date:   Tue Jul 15 11:23:09 2025 -0400

            add cases

        commit 7cb5408c82e9018a083ec5acac2cd147b95f9006
        Author: Saqib Ashraf <[email protected]>
        Date:   Mon Jul 14 16:42:41 2025 -0400

            feat: add ContentType to distinguish between code and string

        commit c71ad7ddc1839e4e4d98e6426956f81bc0b0d2ad
        Author: Saqib Ashraf <[email protected]>
        Date:   Mon Jul 14 12:03:18 2025 -0400

            remove the pill rendering from the fields column

        commit c8a97dc7ca690db53a26a1766863d94e1ca7d8b8
        Author: Saqib Ashraf <[email protected]>
        Date:   Mon Jul 14 12:01:03 2025 -0400

            fix: remove unique pill away from the attribute column. Need to add it to the wording

    commit 278e17381fe3ffafff9e1f30517df5b9351296d7
    Author: Saqib Ashraf <[email protected]>
    Date:   Wed Jul 23 16:24:50 2025 -0400

        POTENTIAL CONFIG CHANGES, PLEASE CONFIRM TO FIX BUILD

    commit 8639f565aafd9195e6cba1dc3bc36925cda8e116
    Author: Saqib Ashraf <[email protected]>
    Date:   Wed Jul 23 16:24:34 2025 -0400

        dictionary

    commit 11e74551d7617f126bccc7ccab38864ebc6fdb9d
    Author: Saqib Ashraf <[email protected]>
    Date:   Wed Jul 23 15:27:21 2025 -0400

        change the dictionary header to match the figma and fix fonting

    commit d8c674104d05efd1a7563a6136ea23bf03ab9c48
    Author: Saqib Ashraf <[email protected]>
    Date:   Wed Jul 23 15:08:44 2025 -0400

        remove uneccssary comments

    commit fac302c57bbe5fa47bdf34f56d9b4f18dd5e715a
    Author: Saqib Ashraf <[email protected]>
    Date:   Wed Jul 23 15:03:10 2025 -0400

        remove all hard coded colors

    commit cb0d7f47baf559b94dc96cca68423fa7f111abd2
    Author: Saqib Ashraf <[email protected]>
    Date:   Wed Jul 23 14:33:24 2025 -0400

        border changes

    commit 9f1317d1c12614eb9a2248ae285af0df44afab84
    Author: Saqib Ashraf <[email protected]>
    Date:   Wed Jul 23 14:26:13 2025 -0400

        fix the bugs inside of the version switcher due to the new types

    commit 19384d9f80a72df3cfb8c1f1aa7a027343d8e4aa
    Author: Saqib Ashraf <[email protected]>
    Date:   Wed Jul 23 13:50:15 2025 -0400

        fix backwards logic

    commit 0304f7efd012cf8173a3dea439878484afe2c3fb
    Author: Saqib Ashraf <[email protected]>
    Date:   Wed Jul 23 13:39:29 2025 -0400

        accordion cleanup

    commit 996ca2d9fb21494ce62a3c862282d2dbfe82ece6
    Author: Saqib Ashraf <[email protected]>
    Date:   Wed Jul 23 11:20:17 2025 -0400

        remove the dictionary download button dependency and add a todo instead

    commit b203fd3b88b8f063808b297cda961204285d87c2
    Author: Saqib Ashraf <[email protected]>
    Date:   Wed Jul 23 11:18:32 2025 -0400

        add ts doc for the accordion component as well as add a TODO for the implementation of the actual dictionary download individual buttons

    commit a2803caea30cced6efdd9da6c9c8d13bd91d9c81
    Author: Saqib Ashraf <[email protected]>
    Date:   Wed Jul 23 11:01:49 2025 -0400

        remove the pages to reduce diff

    commit d28aa3b2b320cede06f687984c952e318dbc3d1c
    Author: Saqib Ashraf <[email protected]>
    Date:   Wed Jul 23 10:59:17 2025 -0400

        merge in the demo branch

commit b2a89083291ea791c37f54bb1f913fb91415c38e
Author: Saqib Ashraf <[email protected]>
Date:   Fri Jul 25 12:37:39 2025 -0400

    Squashed commit of the following:

    commit 6817edf107f7eb02916c00762ae031c555eeba14
    Author: Saqib Ashraf <[email protected]>
    Date:   Fri Jul 25 12:10:46 2025 -0400

        change labeling to make more sense

    commit e00bba0b37a1450b8535c28a317561101add7bc6
    Author: Saqib Ashraf <[email protected]>
    Date:   Fri Jul 25 12:05:08 2025 -0400

        add TODO for dropdown item styling

    commit a0c653b929e9f9a8cf352d258bb28f0ca4f7a7a6
    Author: Saqib Ashraf <[email protected]>
    Date:   Fri Jul 25 11:14:49 2025 -0400

        scroll shadow fix

    commit f618df737aec1161b745ca4c16d29cf24f970066
    Author: Saqib Ashraf <[email protected]>
    Date:   Fri Jul 25 10:55:22 2025 -0400

        Clean up imports as well as licensing and useClipboard inside of the utils

    commit 821fa2bb21f09107cf9d95e819c92eebcbfe509e
    Author: Saqib Ashraf <[email protected]>
    Date:   Fri Jul 25 10:14:54 2025 -0400

        typing the theme for better use and add the licensing

    commit 77ab971dc5b7878963e2f4335b2d35152be7e207
    Merge: 4ef04d0 74a9e02
    Author: Saqib Ashraf <[email protected]>
    Date:   Fri Jul 25 09:57:57 2025 -0400

        Merge branch 'main' into chore/css-fixes

    commit 4ef04d00a06d6b112ad1e4053cdc35cc7a8d3f1f
    Author: Saqib Ashraf <[email protected]>
    Date:   Fri Jul 25 09:56:48 2025 -0400

        fix readmore again

    commit 801a2022abf04f8e9e321c5254197c47b7eeab2a
    Author: Saqib Ashraf <[email protected]>
    Date:   Thu Jul 24 16:24:52 2025 -0400

        added threshold for the readmore text since the line height of the text is distrupting the truncation logic

    commit 07c298f8a14c2cf8ac198f536d133823d21fd9bb
    Author: Saqib Ashraf <[email protected]>
    Date:   Thu Jul 24 16:21:30 2025 -0400

        READ MORE COMPONENT ACTING FUNKY

    commit 1a470c23d3751bc6e25e9e099181b78139e74379
    Author: Saqib Ashraf <[email protected]>
    Date:   Thu Jul 24 16:13:42 2025 -0400

        add licencing

    commit 4dfd1c0ef7df24df18590a45d16e7477b1d61a4f
    Author: Saqib Ashraf <[email protected]>
    Date:   Thu Jul 24 16:13:36 2025 -0400

        add licensing

    commit 6e4511c220b8703d02404fb132f9c9cd45854748
    Author: Saqib Ashraf <[email protected]>
    Date:   Thu Jul 24 15:45:39 2025 -0400

        add TSDoc to componets

    commit a7e06d4c0a394a40d76ec65ac003de2cea98c6a9
    Author: Saqib Ashraf <[email protected]>
    Date:   Thu Jul 24 15:02:43 2025 -0400

        get rid of the functional use of the serialized styles

    commit 74a9e02a702417d582929a27165df3b2b82c0ff8
    Author: Chiso <[email protected]>
    Date:   Thu Jul 24 14:58:14 2025 -0400

        Feat: add display name property to field and Schema  (#331)

        * Add optional displayName field to SchemaFieldBase and Schema

        * Add displayName property to schema definitions and update documentation

        * Add tests for displayName field in SchemaField and Schema

        * ⛏️: remove space in displayName Test

        * cleanup: prettier format for docs alignment

    commit f14e1612623f347f3a0bb64cea0ee8e1e61e0ae6
    Author: Saqib Ashraf <[email protected]>
    Date:   Thu Jul 24 14:07:03 2025 -0400

        fix the bolding

    commit f993fc9694fd84beb7f12109e9228a9da8c87f96
    Author: Saqib Ashraf <[email protected]>
    Date:   Thu Jul 24 13:55:53 2025 -0400

        fix for broken scroll

    commit 63e0d8877da52d1241a8028fa012261159cd5316
    Merge: 48b4b9e 18c35d4
    Author: Saqib Ashraf <[email protected]>
    Date:   Thu Jul 24 13:45:47 2025 -0400

        Merge branch 'main' into chore/css-fixes

    commit 48b4b9ead19572f8dddcb378cf1894b817ada914
    Author: Saqib Ashraf <[email protected]>
    Date:   Thu Jul 24 13:45:27 2025 -0400

        Squashed commit of the following:

        commit 18c35d4c779c6374af18bcec118e9186ed2000f9
        Author: Saqib Ashraf <[email protected]>
        Date:   Thu Jul 24 13:21:20 2025 -0400

            * fix: remove unique pill away from the attribute column. Need to add it to the wording

            * remove the pill rendering from the fields column

            * feat: add ContentType to distinguish between code and string

            * add cases

            * feat: implement all requirements from the compute restrictions table except for handling multiple, foreign Keys.

            * only working in the case of one foreignKey however wil expand in next commit

            * hacked together the multi key version

            * rendering all cases without individually checking all cases inside of the render file

            * hopefully fix issue

            * fix the errors when merging branch over

            * delete unwanted file

            * add back unique after deletion

            * fix naming

            * rename

            * fix the issue for duplicate read more values

            * add complex stories as requested.

            * revert back to old table row.

            * max lines added.

            * hacked together fix for truncation issues.

            * cleanup

            * fix some of the read more text behaviour

            * allow multiple relevant foreign keys

            * field name fix

            * fix with bolding

            * some code cleanup after some suggestions as well as fixing the containers

            * clean up the workout.

            * add an and to the list

            * remove the fragment

            * ui cleanup and logic cleanup

            * fix uniqueness case

            * handle the cases for range

            * add keys

            * changed some of the types and added ts doc

            * remove import of schema

            * variable clean up

            * accidentally forgot to remove this function that is not being used

        commit 1a345657cf2b49b0ca41c8aaac559ad6a2132feb
        Author: Saqib Ashraf <[email protected]>
        Date:   Thu Jul 24 11:52:39 2025 -0400

            add doccumentation for testers, such as the BioInfoTeam (#327)

    commit 4f9a1a5f3b95bb709b3d168434f2f2ca2dba4b70
    Author: Saqib Ashraf <[email protected]>
    Date:   Thu Jul 24 13:44:21 2025 -0400

        Squashed commit of the following:

        commit 18c35d4c779c6374af18bcec118e9186ed2000f9
        Author: Saqib Ashraf <[email protected]>
        Date:   Thu Jul 24 13:21:20 2025 -0400

            * fix: remove unique pill away from the attribute column. Need to add it to the wording

            * remove the pill rendering from the fields column

            * feat: add ContentType to distinguish between code and string

            * add cases

            * feat: implement all requirements from the compute restrictions table except for handling multiple, foreign Keys.

            * only working in the case of one foreignKey however wil expand in next commit

            * hacked together the multi key version

            * rendering all cases without individually checking all cases inside of the render file

            * hopefully fix issue

            * fix the errors when merging branch over

            * delete unwanted file

            * add back unique after deletion

            * fix naming

            * rename

            * fix the issue for duplicate read more values

            * add complex stories as requested.

            * revert back to old table row.

            * max lines added.

            * hacked together fix for truncation issues.

            * cleanup

            * fix some of the read more text behaviour

            * allow multiple relevant foreign keys

            * field name fix

            * fix with bolding

            * some code cleanup after some suggestions as well as fixing the containers

            * clean up the workout.

            * add an and to the list

            * remove the fragment

            * ui cleanup and logic cleanup

            * fix uniqueness case

            * handle the cases for range

            * add keys

            * changed some of the types and added ts doc

            * remove import of schema

            * variable clean up

            * accidentally forgot to remove this function that is not being used

        commit 1a345657cf2b49b0ca41c8aaac559ad6a2132feb
        Author: Saqib Ashraf <[email protected]>
        Date:   Thu Jul 24 11:52:39 2025 -0400

            add doccumentation for testers, such as the BioInfoTeam (#327)

    commit 75b5754d2dab338a8e69f6f8cd5314d8f78a4e97
    Author: Saqib Ashraf <[email protected]>
    Date:   Thu Jul 24 13:41:57 2025 -0400
…
@SaqAsh SaqAsh changed the title Squashed commit of the following: conditional logic modal finalization Jul 29, 2025
@SaqAsh SaqAsh changed the title conditional logic modal finalization #281, 313 - Conditional Logic Modal Implementation Jul 31, 2025
@SaqAsh SaqAsh marked this pull request as ready for review July 31, 2025 20:16
Copy link
Contributor

@joneubank joneubank left a comment

Choose a reason for hiding this comment

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

Great work, thanks. Some requests to clean it up a tiny bit.

@SaqAsh SaqAsh requested a review from joneubank August 1, 2025 20:41
Comment on lines +35 to +51
export type ConditionalLogicModalProps = {
setIsOpen: (isOpen: boolean) => void;
isOpen: boolean;
restrictions: SchemaFieldRestrictions;
currentSchemaField: SchemaField;
};

/**
* Modal component that displays conditional logic restrictions for a schema field
*
* @param setIsOpen - Function to control modal open/close state
* @param isOpen - Current open state of the modal
* @param restrictions - The field restrictions to display
* @param currentSchemaField - The schema field being described
* @returns Modal component with conditional logic visualization
*/
export const ConditionalLogicModal = ({
Copy link
Contributor

Choose a reason for hiding this comment

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

appreciating the typings + comments thank you

@ciaranschutte
Copy link
Contributor

Looking like all the suggested feedback has been implemented - approving

ciaranschutte
ciaranschutte previously approved these changes Aug 7, 2025
Copy link
Contributor

@joneubank joneubank left a comment

Choose a reason for hiding this comment

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

Found a couple small changes that we can include throughout to level up this code before merging.

@SaqAsh SaqAsh requested a review from ciaranschutte August 11, 2025 16:30
Copy link
Contributor

@joneubank joneubank left a comment

Choose a reason for hiding this comment

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

Looks great, ship it.

@SaqAsh SaqAsh merged commit 4ec4b21 into main Aug 12, 2025
2 checks passed
@SaqAsh SaqAsh deleted the feature/conditional-logic-modal branch August 12, 2025 14:38
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