Boer Zhang
Boer Zhang
@zjrwtx @Wendong-Fan Could you please review my PR when you have time? I have modified the code according to Xiaotian's suggestions, and now it is ready for review again! There...
@zjrwtx Hi Yifeng, could you please review my PR when you have time? There is no rush but I want to make sure you have seen this.
@zjrwtx Hi Yifeng, could you please review my PR when you have time? There is no rush but I want to make sure you have seen this.
Hi Yifeng@zjrwtx, no worries at all—I'm grateful to have your feedback! Apologies for missing the conflict earlier. The main repo has changed a bit since my initial PR in February,...
> overall, i think camel/data_collector and camel/messages/conversion can be put in data_schemas folder to make it unify control,what do you think? @boerz-coding @zjrwtx Thank you for the thoughtful suggestion! I...
> > overall, i think camel/data_collector and camel/messages/conversion can be put in data_schemas folder to make it unify control,what do you think? @boerz-coding > > @zjrwtx Thank you for the...
@Wendong-Fan Hi Wendong, this is a gentle reminder on this issue. This PR is ready to be merged. I have already completed the original goal for this refactor, and have...
Thank you @Wendong-Fan for your constructive feedback. I have updated the code accordingly, and now it is ready for review again.
I have moved this PR from my forked repo to the main repo (PR #2362), to allow smooth future collaboration and CI/CD test. I will close this PR for now....