-
Notifications
You must be signed in to change notification settings - Fork 4
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
workflow api's for schedule queries #280
Conversation
…orkflows-new-api
adding comments
…orkflows-new-api
adding tests to workflows fixing tests fixing tests fixing tests fixing tests
b49836c
to
6bc9559
Compare
Hi @nishantmunjal7, thanks for your contribution! 🥳 🎉 I've pushed the refactored changes to this PR, Could you please review these newly added methods manually again on the latest changes 🙏 and let us know if something needs to be updated 😄 Currently, this feature is not enabled on our dev tenant (https://devx8.atlan.com), but we'll be adding integration tests for this feature soon. 💯 |
- Enhanced unit tests. - Code refactored for improved maintainability. - Method names shortened and docstrings added for clarity.
bea4cb6
to
cd2fa1b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have resolved conflicts with the main
branch. Everything looks good as of now, and this feature will be included in the upcoming release 🎉
More info: