bmlt-enabled / bmlt-workflow

supports automation of meeting adds/changes/deletes
Other
4 stars 3 forks source link

Meetings that are being modifed are being set to Unpublished #163

Closed paulnagle closed 1 year ago

paulnagle commented 1 year ago

Hello, The last couple of meetings that have gone through the workflow, have ended up being set to Unpublished.

The only changes that were made were changes to the time or location of the meetings, yet when I check in BMLT after approving the changes in the the workflow, the meetings are Unpublished.

Maybe you need to set published to 1 explicitly in https://github.com/bmlt-enabled/bmlt-workflow/blob/main/src/REST/Handlers/SubmissionsHandler.php#L401

$change['published'] = 1;

but I wasnt able to spend very long looking through the code, so maybe it's something else

nigel-bmlt commented 1 year ago

Could you confirm which version of bmlt you are attaching to?

On Sat, 18 Mar 2023 at 12:42 am, Paul Nagle @.***> wrote:

Hello, The last couple of meetings that have gone through the workflow, have ended up being set to Unpublished.

The only changes that were made were changes to the time or location of the meetings, yet when I check in BMLT after approving the changes in the the workflow, the meetings are Unpublished.

Maybe you need to set published to 1 explicitly in https://github.com/bmlt-enabled/bmlt-workflow/blob/main/src/REST/Handlers/SubmissionsHandler.php#L401

$change['published'] = 1;

— Reply to this email directly, view it on GitHub https://github.com/bmlt-enabled/bmlt-workflow/issues/163, or unsubscribe https://github.com/notifications/unsubscribe-auth/AXW3OUEGHLZ4ICMTC5AC76LW4RS4PANCNFSM6AAAAAAV6RTZQI . You are receiving this because you are subscribed to this thread.Message ID: @.***>

paulnagle commented 1 year ago

BMLT Root Server Version 3.0.1

paulnagle commented 1 year ago

Actually, I am running BMLT version 3.0.3. I upgraded on Feb 24th, and have been seeing the issue since. The workflow plugin is reporting it as 3.0.1 though

image

I have the workflow Plugin Version 1.1.2 installed.

jbraswell commented 1 year ago

Are you actively able to reproduce this issue? If you submit a change to any meeting, it always unpublished it?

paulnagle commented 1 year ago

Yes. I just tried again by slightly changing the name of the group through the workflow plugin, and the meeting became unpublished.

jbraswell commented 1 year ago

@nigel-bmlt can you get me the exact request body and http verb that's being sent to the root server rest api?

paulnagle commented 1 year ago

(chopped debug log) Debug log file attached...

nigel-bmlt commented 1 year ago

Closed now we found the sql driver issue.