Skip to content

new tool to get all schemas #19

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
Merged
merged 18 commits into from
Apr 7, 2025
Merged

new tool to get all schemas #19

merged 18 commits into from
Apr 7, 2025

Conversation

rahulbats
Copy link
Contributor

new tool to get all schemas

rahulbats and others added 15 commits March 13, 2025 18:40
@rahulbats
adding list clusters tool , as this is asked for by my client
143dfa8
@rahulbats
added list clusters as requested by one of my clients
59731d1
@rahulbats
added a list environment and a read environment tool
d8c7654
@rahulbats
chore: remove unused print-md-schema.ts file
f9f0018
@rahulbats
chore: remove redundant clusters.ts file
4b46d71
@rahulbats @eddyv
Update src/confluent/tools/tool-factory.ts
bb0bf7c 
Co-authored-by: Edward Vaisman <[email protected]>
@rahulbats
git checkout -b rahulbats-cluster-operations mainMerge branch 'cluste…
83c61ba 
…r-operations'
@rahulbats
refactor: update environment handlers to use pathBasedClient
ba1531f
@rahulbats
merge: resolve conflicts and complete merge
a4c7f26
@rahulbats
chore: restore print-md-schema.ts file
dd36a00
@rahulbats
chore: update list-clusters-handler and print-md-schema
c6cce2c
@rahulbats
Merge branch 'cluster-operations'
a9c3231
@rahulbats
changes for the schema tool
10ed6c8
@rahulbats
Merge main branch and resolve conflicts
cca7d64
@rahulbats
Add schema registry list handler
4fcfff9
@Copilot Copilot AI review requested due to automatic review settings April 4, 2025 18:51
@rahulbats rahulbats requested a review from a team as a code owner April 4, 2025 18:51
Copilot
Copilot AI reviewed Apr 4, 2025
View reviewed changes
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

eddyv
eddyv reviewed Apr 4, 2025
View reviewed changes
rahulbats
rahulbats commented Apr 5, 2025
View reviewed changes
Copy link
Contributor Author

@rahulbats rahulbats left a comment

Choose a reason for hiding this comment

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

changed the error message and showing the postman screenshot of why we need the latestOnly

@eddyv eddyv self-requested a review April 7, 2025 13:03
eddyv
eddyv requested changes Apr 7, 2025
View reviewed changes
Copy link
Member

@eddyv eddyv left a comment

Choose a reason for hiding this comment

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

see #19 (comment)

@eddyv eddyv self-requested a review April 7, 2025 14:21
@eddyv eddyv merged commit 21ceed7 into confluentinc:main Apr 7, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

Copilot code review Copilot Copilot left review comments

@eddyv eddyv eddyv approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@rahulbats @eddyv