-
Notifications
You must be signed in to change notification settings - Fork 17
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
Update to 5.x #338
base: 5.x
Are you sure you want to change the base?
Update to 5.x #338
Conversation
…t-items-request Maut-10760: fix custom object field values for api request
Fixes for PHPStan & PHP CS Fixer
Update README.md
Remove PUT from the available item operations for the custom_objects endpoint
Fix test case
Of course I can. I checked, there are a few conflicts. So I should be able to do it in the next few days. I apologize, sometimes I don't have time, so I don't respond immediately. If someone can do it faster, it's ok. |
@mehransobhani Could you please check which branch do you use. The
And could you please check if there is a method |
@volha-pivavarchyk |
It should be fixed in Mautic using the |
I'm trying to get the acquia/mc-cs-plugin-custom-objects plugin working on Mautic v5.1.0 but keep encountering errors. Any suggestions or an estimated timeline for a fix? |
When this PR is tested and merged, it'll be compatible with the latest release, if you can help with testing / fixing issues reported, that will help move it along. |
…filter value is "yesterday" and operator is "gt" for date field (acquia#359) * Fix custom parameter date issue * add more test case
@escopecz I tried merging changes from the staging branch, resolved conflicts, then some tests were broken. To be honest, I did this week ago and haven't had time to fix all the broken tests. Just a few of them. Because it takes more than a few hours. I guess I won't be able to get back to it until two weeks from now. But I didn't see any changes in the Github Action configuration after the merge. So maybe the settings are the reason why the CI job for the test is not running. For example, branch naming, PR from forks. Can anyone check these settings as I am unable to do so. If yes, we can run this job for these changes and add changes from staging later. |
@escopecz I have fixed 4 more tests. But there are still failed tests after the merge. Fixing them will take time, as it's due to big updates in the staging branch, which are not in this branch. I won't be able to get back to it until a week and a half from now. Do I need to push the merge and commits with the fixed tests after merging? Or can this PR be without the merge in this branch? |
@volha-pivavarchyk it doesn't seem that the merge was complete. The .github/workflows folder is missing from this branch. I don't fully understand what you are saying. You shared a screenshot that the merge commit is in the history. I thought you pushed it but now I checked it's not pushed. So the GH Actions won't run. Why do you think you should not push it? |
@escopecz I merged staging and fixed all tests. Works well for me. What next? I apologize for my oversight. I thought that the workflows directory was in the project. |
@volha-pivavarchyk great news! I triggered the build and there seem to be some issue. Can you take a look? I will test this once the CI is passing. |
@escopecz I updated dependencies and workflow file (removed old php version and use new mautic version (v5.1)). Could you please trigger CI again. |
Thanks! Triggered. |
Simplified Guide: Installing Custom Objects Plugin for Mautic 5.1.1 Al following steps are done locally in your computer before you even go to mautic instance. Step 1: Clone the Plugin and Apply Compatibility Changes
Step 2: Prepare the Plugin for Upload
Step 3: Upload and Extract on Mautic Server
Step 4: Install the Plugin in Mautic
IMPORTANT STEP: I had error 500, to fix it you need to manually run this command:
o You should see output confirming that 1 plugin was installed.
Step 5: Verify Plugin Installation
Final Notes
After all this you will have CustomObjectsBundle running for Mautic 5 version. |
@escopecz I fixed the problems after analyzing the cs fixer program. So far, three (cs, twig, phpunit) of the five tasks are working fine locally. Phpstan and rector failed. Do I need to fix all of them? Or is it possible to create a baseline for phpstan for example? |
@volha-pivavarchyk yes, let's generate a baseline file for the PHPSTAN issues. |
Description:
Replaces PRs #298, #308, #316
Steps to test this PR: