e2corporation opened a new pull request, #2928: URL: https://github.com/apache/incubator-devlake/pull/2928
### ⚠️ Config-UI / Data Integrations / Connection Manager - [ ] `Fix` Connection Save Handling and Remove extra `savePromise` added during recent refactor - [ ] `Fix` Restore Successful Toast Notifications on Connection Save - [ ] `Fix` Remove `setTimeout` Handler - [ ] `Fix` Remove legacy inlined connection test post-save - [ ] `Feat` Add **Connection** _Data Model_ (`models/Connection.js`) - [ ] `Refactor` Change legacy uppercase Connection `ID` prop to `id` - [ ] `Refactor` change Connection Prop `rateLimit` to `rateLimitPerHour` for future consistency - [ ] `Fix` Resolve Lint warnings with missing React Effect Dependencies ### Description This PR performs refactoring enhancements to **Connection Manager Hook** that restore the desired save handling workflow for data connections. After a connection is successfully saved, the user is redirected back to the Manage Connections screen _only_ when Adding a New Connection. When Modifying an existing Connection, the user will remain on the Configure Connection screen after saving. ### Does this close any open issues? Closes #2923 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
