Implement the Plugin sql updates #842

Closed
saman222 wants to merge 1 commits from staging into staging
saman222 commented 1 year ago (Migrated from github.com)
Owner

I checked the JCB codes and tried to implement the plugin sql updates section , which did not work , according to the JCB structure

I checked the JCB codes and tried to implement the plugin sql updates section , which did not work , according to the JCB structure
Owner

I have looked over the code, and for the most part you did a great job... but I think there is a simpler way, that does not need us
messing with some of the function you touched.

I am a little over occupied at the moment, but will give it another look the second or third week or so in the new year.

I have looked over the code, and for the most part you did a great job... but I think there is a simpler way, that does not need us messing with some of the function you touched. I am a little over occupied at the moment, but will give it another look the second or third week or so in the new year.
saman222 commented 1 year ago (Migrated from github.com)
Owner

I'm trying to get a professional membership. Do you accept this commit of mine as my second commit, regardless of whether you apply it or not?
I spend many time to do this commit.
And i want to go for 3rd commit. Have you any reformable code suggest?

I'm trying to get a professional membership. Do you accept this commit of mine as my second commit, regardless of whether you apply it or not? I spend many time to do this commit. And i want to go for 3rd commit. Have you any reformable code suggest?
Owner

I will in good faith give you access trusting that you will continue to help and contribute to JCB ;) happy new year!

I will in good faith give you access trusting that you will continue to help and contribute to JCB ;) happy new year!
saman222 commented 1 year ago (Migrated from github.com)
Owner

Thank you very much @Llewellynvdm .
I was looking forward to this moment. I will definitely try to play an active role.

Thank you very much @Llewellynvdm . I was looking forward to this moment. I will definitely try to play an active role.
saman222 commented 1 year ago (Migrated from github.com)
Owner

hi @Llewellynvdm
if you give me a little guidance, i can implement Plugin sql updates with your standard path and code style.

hi @Llewellynvdm if you give me a little guidance, i can implement Plugin sql updates with your standard path and code style.
This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: joomla/Component-Builder#842
Loading…
There is no content yet.