Skip to content
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

Do not process already processing feeds #63

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

arjanfrans
Copy link

#62

@igormukhingmailcom
Copy link
Member

Thanks for PR, @arjanfrans

But the proper solution would be to create and use a new repository method as described here:
#62 (comment)

@arjanfrans
Copy link
Author

Thanks for PR, @arjanfrans

But the proper solution would be to create and use a new repository method as described here:

#62 (comment)

Makes sense, I'll update that.

@arjanfrans
Copy link
Author

@igormukhingmailcom

I pushed an update. I am just wondering, if the changes are okay; will you be able to merge it and release it anytime soon?

@pschuler21
Copy link

@igormukhingmailcom

I would also be interested in getting this fixed. The changes of @arjanfrans would make your plugin more useable for our project. Any chance for you to merge and release in the near future?

@pschuler21
Copy link

@igormukhingmailcom there was a release a few days ago without the changes of @arjanfrans , is there a specific reason that keeps you from merging this PR?

@loevgaard
Copy link
Member

Can you rebase this PR onto master? :)

@arjanfrans arjanfrans force-pushed the do-not-process-processing-feeds branch from f963786 to 9efab8c Compare June 30, 2022 12:47
@arjanfrans
Copy link
Author

Can you rebase this PR onto master? :)
done

@loevgaard
Copy link
Member

Do you have time to look into the build errors, @arjanfrans? :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants