Skip to content

flow type structure #486

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

Open
wants to merge 54 commits into
base: main
Choose a base branch
from
Open

flow type structure #486

wants to merge 54 commits into from

Conversation

Knerio
Copy link
Member

@Knerio Knerio commented May 28, 2025

close #500
close #420
close #388
close #475
close #401
close #462

@Knerio Knerio self-assigned this May 28, 2025
Copy link

github-actions bot commented May 28, 2025

GitLab Pipeline Action

General information

Link to pipeline: https://gitlab.com/code0-tech/development/sagittarius/-/pipelines/1856819960

Status: Passed
Duration: 4 minutes

Job summaries

rspec: [ee]

Coverage report available at https://code0-tech.gitlab.io/-/development/sagittarius/-/jobs/10274492742/artifacts/tmp/coverage/index.html
Test summary available at https://gitlab.com/code0-tech/development/sagittarius/-/pipelines/1856819960/test_report
Finished in 18.86 seconds (files took 12.97 seconds to load)
1131 examples, 0 failures
3624 / 4037 LOC (89.77%) covered.
[TEST PROF INFO] Time spent in factories: 00:11.230 (43.2% of total time)

rspec: [ce]

Coverage report available at https://code0-tech.gitlab.io/-/development/sagittarius/-/jobs/10274492741/artifacts/tmp/coverage/index.html
Test summary available at https://gitlab.com/code0-tech/development/sagittarius/-/pipelines/1856819960/test_report
Finished in 19.85 seconds (files took 12.28 seconds to load)
1084 examples, 0 failures
3471 / 3900 LOC (89.0%) covered.
[TEST PROF INFO] Time spent in factories: 00:11.093 (43.36% of total time)

rubocop

615 files inspected, no offenses detected

@Knerio Knerio force-pushed the 388-update-delete-create-flow branch from 3a70731 to 325ed59 Compare May 29, 2025 00:39
@Knerio Knerio force-pushed the 420-flow-type-structure branch from 457223f to 1e5b28e Compare May 29, 2025 00:42
@Knerio Knerio changed the base branch from 388-update-delete-create-flow to main May 29, 2025 18:39
@Knerio Knerio force-pushed the 420-flow-type-structure branch from 354ef27 to 3c6b199 Compare May 29, 2025 18:42
@Knerio Knerio force-pushed the 420-flow-type-structure branch 2 times, most recently from de3ee11 to d002ae0 Compare May 30, 2025 13:01
@Knerio Knerio changed the base branch from main to 491-type-graphql-ids May 30, 2025 20:58
@Knerio Knerio force-pushed the 420-flow-type-structure branch 2 times, most recently from b71f4d7 to 8054e3e Compare May 30, 2025 22:31
Base automatically changed from 491-type-graphql-ids to main May 30, 2025 22:44
@Knerio Knerio force-pushed the 420-flow-type-structure branch from 387a60a to a09d8b2 Compare May 31, 2025 10:55
@Knerio Knerio requested a review from Taucher2003 May 31, 2025 12:46
@Knerio Knerio mentioned this pull request May 28, 2025
@Knerio Knerio marked this pull request as ready for review May 31, 2025 23:57
@Knerio Knerio changed the title WIP: flow type structure flow type structure Jun 1, 2025
return if values.count(true) == 1

errors.add(:base,
'Exactly one of literal_value, reference_value, or function_value must be present')
Copy link
Member

Choose a reason for hiding this comment

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

The string might make translation in the frontend difficult

Copy link
Member Author

Choose a reason for hiding this comment

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

I dont think thats a problem here. The frontend should send invalid data to us in the first place. This case is a very easy thing to validate in the frontend

@Knerio Knerio force-pushed the 420-flow-type-structure branch from 5faca64 to b836858 Compare June 6, 2025 08:37
@Knerio Knerio requested a review from Taucher2003 June 6, 2025 10:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants