8000 TA-3725: Add list command options to filter by variable value and type. by jetakasabaqi · Pull Request #196 · celonis/content-cli · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
/ content-cli Public < 8000 /div>

TA-3725: Add list command options to filter by variable value and type. #196

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

TA-3725: Add list command options to filter by variable value and type. #196

wants to merge 10 commits into from

Conversation

jetakasabaqi
Copy link
Contributor
@jetakasabaqi jetakasabaqi commented May 7, 2025 •
edited
Loading

Description

Add variableValue and variableType options to the list command.
Added the implementation of calling findPackagesByVariableValue if variableValue is sent.

PR will be merged after the endpoint is released on 19 May.

Checklist

  • I have self-reviewed this PR
  • I have tested the change and proved that it works in different scenarios
  • I have updated docs if needed

@jetakasabaqi jetakasabaqi marked this pull request as ready for review May 7, 2025 15:26
@jetakasabaqi jetakasabaqi requested review from GencBlakqoriP9 and a team as code owners May 7, 2025 15:26
Buqeta
Buqeta reviewed May 7, 2025
View reviewed changes
Copy link
Contributor
@Buqeta Buqeta left a comment •
edited
Loading

Choose a reason for hiding this comment

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

We also need to update the content cli version on the package.json file.

LaberionAjvazi
LaberionAjvazi reviewed May 8, 2025
View reviewed changes
jetakasabaqi and others added 3 commits May 8, 2025 11:15
@jetakasabaqi @LaberionAjvazi
Update src/commands/config.command.ts
7d71f19 
Co-authored-by: Laberion Ajvazi <93651741+LaberionAjvazi@users.noreply.github.com>
@jetakasabaqi @LaberionAjvazi
Update src/commands/config.command.ts
f755b5d 
Co-authored-by: Laberion Ajvazi <93651741+LaberionAjvazi@users.noreply.github.com>
@jetakasabaqi @LaberionAjvazi
Update src/api/batch-import-export-api.ts
dc2f77b 
Co-authored-by: Laberion Ajvazi <93651741+LaberionAjvazi@users.noreply.github.com>
ZgjimHaziri
ZgjimHaziri previously approved these changes May 8, 2025
View reviewed changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ZgjimHaziri ZgjimHaziri ZgjimHaziri approved these changes

@promeris promeris promeris approved these changes

@Buqeta Buqeta Buqeta approved these changes

@LaberionAjvazi LaberionAjvazi LaberionAjvazi approved these changes

@ksalihu ksalihu ksalihu approved these changes

@GencBlakqoriP9 GencBlakqoriP9 Awaiting requested review from GencBlakqoriP9 GencBlakqoriP9 is a code owner

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@jetakasabaqi @ZgjimHaziri @promeris @Buqeta @LaberionAjvazi @ksalihu
0