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

feat: add is_statutory_application_type to published_flows table #4169

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

RODO94
Copy link
Contributor

@RODO94 RODO94 commented Jan 20, 2025

What does this PR do?

To enable users to see which services are Statutory we need to define this in the flow data. The approach I have taken is to add a check when a service is published to see if any Question || Checklist || SetValue component has an fn = "application.type" and then cross reference this value with the ODP Schema using getValidSchemaValues("ApplicationType");

This will enable all new published services to be defined as Statutory or not. It will not automatically ascribe a non-statutory service as "Discretionary" service as a service may be a modular service used to build other services, making it neither Discretionary or Statutory

Copy link

github-actions bot commented Jan 20, 2025

🤖 Hasura Change Summary compared a subset of table metadata including permissions:

Updated Tables (1)

  • public.published_flows permissions:

    insert select update delete
    api / /
    demoUser / /
    platformAdmin / /
    public /
    teamEditor / /
    9 added column permissions
    insert select update
    api ➕ is_statutory_application_type ➕ is_statutory_application_type
    demoUser ➕ is_statutory_application_type
    platformAdmin ➕ is_statutory_application_type
    public
    teamEditor ➕ is_statutory_application_type

@RODO94 RODO94 self-assigned this Jan 20, 2025
Copy link

Pizza

Deployed 6a10712 to https://4169.planx.pizza.

Useful links:

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.

1 participant