-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
feat: added sorting for name, connector and lastsync field #12302
Conversation
@timroes can you review this when you get time? |
Tested locally and feature works as expected 👍 |
@timroes can we merge the PR? |
@dizel852 usually we have it on the author (if a member of airbyte) to merge that PR. Given that this PR is now rather outdated with master, I'd suggest we bring it once more up to date with master and have CI pass on it before merging. |
@timroes gotcha gotcha 😄 |
a7b5cfb
to
bf52122
Compare
Hey I have rebased it with latest master @timroes is it ok if I merge now |
Good to go, since you got your reviews :) |
* feat: added sorting for name, connector and lastsync field * fix: remove console.log
What
closes #8754