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

fix: typo in minilang query field spec and column map #2605

Merged
merged 3 commits into from
Aug 1, 2024

Conversation

fregataa
Copy link
Member

@fregataa fregataa commented Aug 1, 2024

Checklist: (if applicable)

  • Milestone metadata specifying the target backport version
  • API server-client counterparts (e.g., manager API -> client SDK)

@fregataa fregataa added this to the 23.09 milestone Aug 1, 2024
@fregataa fregataa requested a review from lizable August 1, 2024 08:46
@fregataa fregataa self-assigned this Aug 1, 2024
@github-actions github-actions bot added comp:manager Related to Manager component size:XS ~10 LoC labels Aug 1, 2024
@fregataa fregataa changed the title fix: mistypo of minilang field spec & column map fix: mistype of minilang field spec & column map Aug 1, 2024
@fregataa fregataa changed the title fix: mistype of minilang field spec & column map fix: mistype in minilang query field spec and column map Aug 1, 2024
@fregataa fregataa changed the title fix: mistype in minilang query field spec and column map fix: typo in minilang query field spec and column map Aug 1, 2024
@fregataa fregataa added this pull request to the merge queue Aug 1, 2024
Merged via the queue into main with commit 7bf1835 Aug 1, 2024
20 checks passed
@fregataa fregataa deleted the fix/mistypo-of-minilang branch August 1, 2024 08:58
lablup-octodog pushed a commit that referenced this pull request Aug 1, 2024
Backported-from: main (24.09)
Backported-to: 23.09
Backport-of: 2605
lablup-octodog pushed a commit that referenced this pull request Aug 1, 2024
Backported-from: main (24.09)
Backported-to: 24.03
Backport-of: 2605
github-merge-queue bot pushed a commit that referenced this pull request Aug 1, 2024
github-merge-queue bot pushed a commit that referenced this pull request Aug 6, 2024
agatha197 pushed a commit to lablup/backend.ai-webui that referenced this pull request Aug 8, 2024
### TL;DR

Added support for boolean options in BAIPropertyFilter component.

![image.png](https://graphite-user-uploaded-assets-prod.s3.amazonaws.com/XqC2uNFuj0wg8I60sMUh/70f8c94e-fffe-4e20-bddf-b1e5f530ab1b.png)

Related core PR: lablup/backend.ai#2605

### What changed?

1. `AgentList.tsx`: Added `type` and `options` properties to filter properties.
2. `BAIPropertyFilter.stories.tsx`: Updated story to include boolean type property.
3. `BAIPropertyFilter.tsx`: Major updates to support boolean options including default boolean filter options and enhanced value handling.
4. Updated i18n files for new labels.

### How to test?

1. Go to the AgentList component and verify the filter functionality with boolean options.
2. Check the storybook for BAIPropertyFilter and ensure the new boolean property works as expected.
3. Verify i18n changes reflect correctly in the UI.

### Why make this change?

To enhance the filtering capabilities by allowing boolean options, providing more flexibility and utility in data filtering.

---

<!--
Please precisely, concisely, and concretely describe what this PR changes, the rationale behind codes,
and how it affects the users and other developers.
-->

**Checklist:** (if applicable)

- [ ] Mention to the original issue
- [ ] Documentation
- [ ] Minium required manager version
- [ ] Specific setting for review (eg., KB link, endpoint or how to setup)
- [ ] Minimum requirements to check during review
- [ ] Test case(s) to demonstrate the difference of before/after
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp:manager Related to Manager component size:XS ~10 LoC
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant