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

The poller is now the worker and we have a "scripts" folder #3920

Merged
merged 4 commits into from Nov 19, 2017

Conversation

5 participants
@annando
Collaborator

annando commented Nov 19, 2017

Hopefully this will not affect the ongoing changes from @MrPetovan

All executable scripts are now moved out of the include folder.

Additionally all occurrences of "poller.php" are renamed to "worker.php" since the scripts doesn't do any polling for some time now.

@tobiasd

This comment has been minimized.

Show comment
Hide comment
@tobiasd

tobiasd Nov 19, 2017

Collaborator

Mean everybody has to change their cron jobs?

Collaborator

tobiasd commented Nov 19, 2017

Mean everybody has to change their cron jobs?

@annando

This comment has been minimized.

Show comment
Hide comment
@annando

annando Nov 19, 2017

Collaborator

Yeah, this will be fun :-)

Collaborator

annando commented Nov 19, 2017

Yeah, this will be fun :-)

@AlfredSK

This comment has been minimized.

Show comment
Hide comment
@AlfredSK

AlfredSK Nov 19, 2017

Oh! The cronjob has to be changed to meet the new path and file name. Right? So, I better disable my nightly cronjob for git-pulling from develop to be able to make both changes at once.

AlfredSK commented Nov 19, 2017

Oh! The cronjob has to be changed to meet the new path and file name. Right? So, I better disable my nightly cronjob for git-pulling from develop to be able to make both changes at once.

@MrPetovan

This comment has been minimized.

Show comment
Hide comment
@MrPetovan

MrPetovan Nov 19, 2017

Collaborator

Yay! This makes way better sense, I can't believe I managed to talk you into actually doing it 😋

Collaborator

MrPetovan commented Nov 19, 2017

Yay! This makes way better sense, I can't believe I managed to talk you into actually doing it 😋

@MrPetovan

This comment has been minimized.

Show comment
Hide comment
@MrPetovan

MrPetovan Nov 19, 2017

Collaborator

Pulling back from merging until @AlfredSK is ready.

Collaborator

MrPetovan commented Nov 19, 2017

Pulling back from merging until @AlfredSK is ready.

@AlfredSK

This comment has been minimized.

Show comment
Hide comment
@AlfredSK

AlfredSK Nov 19, 2017

😂 cronjob for updates disabled. Good night.

AlfredSK commented Nov 19, 2017

😂 cronjob for updates disabled. Good night.

@MrPetovan MrPetovan merged commit 81647ce into friendica:develop Nov 19, 2017

@annando annando deleted the annando:scripts branch Nov 19, 2017

@zeroadam

This comment has been minimized.

Show comment
Hide comment
@zeroadam

zeroadam Nov 19, 2017

Should there be a support forum post about this?

zeroadam commented Nov 19, 2017

Should there be a support forum post about this?

@annando

This comment has been minimized.

Show comment
Hide comment
@annando

annando Nov 19, 2017

Collaborator

You have the job.

Collaborator

annando commented Nov 19, 2017

You have the job.

@MrPetovan MrPetovan added this to the 3.6 milestone Mar 9, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment