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

Pass bools as strings in helm chart #13209

Merged
merged 1 commit into from
Jul 22, 2024
Merged

Conversation

mirkoSekulic
Copy link
Collaborator

Description

Related Issue(s)

  • #{issue number}

Verification

  • Your code builds clean without any errors or warnings
  • Manual testing done (required)
  • Relevant automated test added (if you find this hard, leave it and we'll help out)

Documentation

  • User documentation is updated with a separate linked PR in altinn-studio-docs. (if applicable)

@mirkoSekulic mirkoSekulic changed the title pass bools as strings Pass bools as strings in helm chart Jul 22, 2024
Copy link
Contributor

@mlqn mlqn left a comment

Choose a reason for hiding this comment

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

👍

@mirkoSekulic mirkoSekulic merged commit 4d414b0 into main Jul 22, 2024
1 check passed
@mirkoSekulic mirkoSekulic deleted the hotfix/eid-logger-deploy-fix branch July 22, 2024 13:28
Jondyr pushed a commit that referenced this pull request Jul 22, 2024
@mirkoSekulic mirkoSekulic added the skip-releasenotes Issues that do not make sense to list in our release notes label Jul 22, 2024
Jondyr added a commit that referenced this pull request Jul 29, 2024
)

* Change `AppPreviewBar` LegacySelect to StudioNativeSelect

* Change `SecurityLevelSelect` LegacySelect to StudioNativeSelect

* Change `LangSelector` LegacySelect to StudioNativeSelect

* Remove selected property from <option>

* Change `SchemaInspector` LegacySelect to StudioNativeSelect

* Change `PolicyCardRules` LegacySelect to StudioNativeSelect

* Use defaultValue instead of selected on <option>

* Change `TextResourceEditor` LegacySelect to StudioNativeSelect

* Change `componentSpecificContent` LegacySelect to StudioNativeSelect

* Delete unused code

* Change `componentconfig` LegacySelect to StudioNativeSelect

Remove unused import

* Fix some tests broken after LegacySelect->StudioNativeSelect swap

Fix some tests broken after LegacySelect->StudioNativeSelect swap

* Fix failing `each()` tests

* Replace LegacyTextField with StudioTextfield in schemainspector

* Fix css after replacing legacy components

* Replace LegacyTextField and fix possible bug with saving float numbers

Bug: saving float numbers for min/max restrictions on number components

* Fix formatting

* Replace LegacyTextField for ux-editor, delete unused code

* Fix tests after replacing LegacyTextField

* Fix interesting test failure, new component allows exponents like 1e2

"1test 2" becomes "1e2" = 1*10^2=100

* Fix tests using wrong role for number input

* Widen test for coverage

* Add tests for combobox enum with multiple selection

Remove unused code

Fix accidental deletion of test value

* Fix formatting, prettier

* Merge main test fixes

* Fix playwright test failing after legacy component replacement

* fixed overfloee in datamodellknytninger accordion

* fixes overflow in datamodel bilnding in featureFlag

* Replace LegacyPopover for ReleaseContainer

* Replace LegacyPopover for datamodel deletion button

* Replace LegacyPopover for AltinnConfirmationDialog

* Change AltinnConfirmationDialogProps to use StudioPopoverProps

* Replace LegacyPopover for ThreeDotsMenu

* Replace LegacyPopover in PreviewLimitationsInfo

* Replace LegacyTable for TextList

* Fix click-and-hold scrolling in designer (#13194)

* Fix dashboard URL in Playwright (#13203)

* eid logger deploy env setup (#13205)

* Update dependency org.springframework.boot:spring-boot-starter-parent to v3.3.2 (#13198)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* change helm repository (#13208)

* pass bools as strings  in helm chart(#13209)

* Add missing ids to nativeselect, required for label `for`

* Fix playwright tests failing after LegacySelect replacement

Also change value assertion to follow playwright best practices:

**NOTE** If you need to assert input value, prefer
[expect(locator).toHaveValue(value[,
options])](https://playwright.dev/docs/api/class-locatorassertions#locator-assertions-to-have-value)
to avoid flakiness. See [assertions
guide](https://playwright.dev/docs/test-assertions) for more details.

Fix typo

* Make sure ids for tablerowtype is unique in a multi type list

* Replace LegacyToggleButtonGroup on the preview page

* Fix app preview tests for togglegroup

button -> radio

* Replace legacyfieldset for mapcomponent

* Replace LegacyTextArea in ItemDataComponent

* Update import from storybook entry for resizablelayout

* Replace legacy components for ux-editorv3

* Remove design-system-react dependency

Squash: Also commit `yarn.lock`

* Fix linting issues

* Update SelectDataModelComponent.test.tsx

* Fix informative message variant validation issue

* Fix some event handling collisions between new and old components

* Remove commented unused css

* Merge main

* Use `useId` hook to generate unique fallback id for StudioNativeSelect

* Revert change to button disabled property check in langselector test

* Add name option back to getRole query

---------

Co-authored-by: JamalAlabdullah <jamal.mustafa.alabdullah@digdir.no>
Co-authored-by: Erling Hauan <148075168+ErlingHauan@users.noreply.github.com>
Co-authored-by: Tomas Engebretsen <tomas.engebretsen@digdir.no>
Co-authored-by: Mirko Sekulic <misha.sekulic@gmail.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip-releasenotes Issues that do not make sense to list in our release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants