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

Release #1030

Merged
merged 61 commits into from
Feb 13, 2025
Merged

Release #1030

merged 61 commits into from
Feb 13, 2025

Conversation

vytisbulkevicius
Copy link
Contributor

@vytisbulkevicius vytisbulkevicius commented Feb 6, 2025

  • Share the link in the first comment [PRO]
  • Adds action scheduler support
  • Fixed issue with remaining accounts being temporarily invisible in the Post Form tab after removing any existing account
  • Added compatibility with Polylang and Translatepress plugins [PRO]
  • Fixed issue with image not being sent via Zapier integrations
  • Added Mastodon network service [PRO]
  • Added Telegram network service [PRO]

preda-bogdan and others added 30 commits April 4, 2024 18:20
Bumps [phpunit/phpunit](https://github.com/sebastianbergmann/phpunit) from 5.7.27 to 9.6.19.
- [Release notes](https://github.com/sebastianbergmann/phpunit/releases)
- [Changelog](https://github.com/sebastianbergmann/phpunit/blob/9.6.19/ChangeLog-9.6.md)
- [Commits](sebastianbergmann/phpunit@5.7.27...9.6.19)

---
updated-dependencies:
- dependency-name: phpunit/phpunit
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Bumps [codeinwp/themeisle-sdk](https://github.com/Codeinwp/themeisle-sdk) from 3.3.37 to 3.3.40.
- [Release notes](https://github.com/Codeinwp/themeisle-sdk/releases)
- [Changelog](https://github.com/Codeinwp/themeisle-sdk/blob/v3.3.40/CHANGELOG.md)
- [Commits](Codeinwp/themeisle-sdk@v3.3.37...v3.3.40)

---
updated-dependencies:
- dependency-name: codeinwp/themeisle-sdk
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
vytisbulkevicius and others added 7 commits February 6, 2025 15:08
Fixed empty featured image issue
Added mastodon network service
…t/codeinwp/themeisle-sdk-3.3.40

chore(deps): bump codeinwp/themeisle-sdk from 3.3.37 to 3.3.40
Save post sharing logs to ROP API
Added telegram network service
@vytisbulkevicius
Copy link
Contributor Author

@girishpanchal30,

I tested the release and encountered one problem related to: https://github.com/Codeinwp/tweet-old-post-pro/issues/564
I already merged this PR: https://github.com/Codeinwp/rop-auth-service/pull/95 and tested with this Release.

Data is successfully saved in the logs when I share on networks that we had before but since you also added Telegram and Mastodon please add the code to log data when sharing on those new networks as well.

@girishpanchal30
Copy link
Contributor

@vytisbulkevicius I’ve implemented it for Telegram and Mastodon. Please review and merge it into the development branch.

PR: #1031

@vytisbulkevicius vytisbulkevicius merged commit 11b229b into master Feb 13, 2025
7 checks passed
@pirate-bot
Copy link
Contributor

🎉 This PR is included in version 9.2.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@pirate-bot pirate-bot added the released Indicate that an issue has been resolved and released in a particular version of the product. label Feb 13, 2025
@vytisbulkevicius
Copy link
Contributor Author

@girishpanchal30,

For me the deployment to to wordpress.org is failing in this action: https://github.com/Codeinwp/tweet-old-post/actions/runs/13310266940/job/37173728635

Looks like it timeouts after trying to add many files from dependency: /vendor/voku/portable-ascii/src/voku/helper/data/

Do you know if this is something what was included with the latest changes and maybe you have a recommendation on how to overcome this?

@girishpanchal30
Copy link
Contributor

@vytisbulkevicius It looks like there's an issue with SVN. Could you please retry triggering the action?
Error: svn: E000104: Error running context – Connection reset by peer.

@vytisbulkevicius
Copy link
Contributor Author

@girishpanchal30,
Yes, we retried yesterday but it still failed after running about 30 minutes. Re-trigged today and same happened, also after about 30 minutes of running. Any other ideas?

@girishpanchal30
Copy link
Contributor

@vytisbulkevicius Make sure the credentials are valid, as the issue might be related to recent WordPress SVN password changes.

Ref: https://make.wordpress.org/meta/handbook/tutorials-guides/svn-access/

@girishpanchal30 girishpanchal30 mentioned this pull request Feb 14, 2025
6 tasks
@girishpanchal30
Copy link
Contributor

@vytisbulkevicius It looks like there was an issue with the Telegram SDK, so I removed it and switched to the wp_remove_* function to resolve it.

Please test this PR and let me know if you find any issues.

@vytisbulkevicius
Copy link
Contributor Author

@girishpanchal30,

I tried to connect using this PR together with Release PRO -
https://github.com/Codeinwp/tweet-old-post-pro/pull/572

And it doesn't work for me, are you able to connect to Telegram using it?
Here is my test instance:
Admin area URL: https://thoughtlessprocess.s3-tastewp.com/wp-admin
Username: bvytis
Password: FmGD4ZrWNB4

image

@girishpanchal30
Copy link
Contributor

And it doesn't work for me, are you able to connect to Telegram using it?

@vytisbulkevicius Yes, I tested it locally, and it worked fine for me. I'm now checking your test instance.

@girishpanchal30
Copy link
Contributor

@vytisbulkevicius I able to connect the account in your test instance as well.

image

@vytisbulkevicius
Copy link
Contributor Author

That's weird, for me it still does not work. I tried creating a new test instance and still facing the same error.

While if I use the previous Release build: #1030 (comment)

it works just fine and I can connect.

I'm trying to use both the channel ID or handle..

@ReviveSocialChannel
7703794849:AAHPFJce756ULtgUCjVC_jYqQxZio5inszQ

If you use these you still can connect? I wonder if it is something about the location I'm connecting from or what could it be

@girishpanchal30
Copy link
Contributor

girishpanchal30 commented Feb 14, 2025

@vytisbulkevicius It seems issue with PHP 8 when profile photo is not set in telegram user account, now I've fixed it with the latest commit.

@vytisbulkevicius
Copy link
Contributor Author

Yes, it works now, thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released Indicate that an issue has been resolved and released in a particular version of the product.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants