wordpress-mobile / AztecEditor-Android

A reusable native Android rich text editor component.
Mozilla Public License 2.0
675 stars 112 forks source link

Failed Build #951

Closed Kodak1234 closed 2 years ago

Kodak1234 commented 2 years ago

The latest release build is failing, check jitpack.io. So unable to import project into android studio

Kodak1234 commented 2 years ago

image

hypest commented 2 years ago

Thanks for reporting!

Here's the build log on the JitPack job:

* What went wrong:
Execution failed for task ':picasso-loader:prepareToPublishToS3'.
> Either tagName or both branch name and sha1 command line arguments needs to be provided!
  Example usages:
  --tag-name={tag-name}
  --pull-request-number={pull-request-number} --branch-name={branch-name} --sha1={sha1-commit-hash}
  --branch-name={branch-name} --sha1={sha1-commit-hash}

👋 @oguzkocer , would you have the chance to have a look? Looks like a failure that happens on multiple (older) releases, not sure if this is due to some recent change or not. Thanks!

hypest commented 2 years ago

My ping above probably didn't go through (it was erroneously left inside the code block) so, let me add this new comment with it. 👋 @oguzkocer , can you have a look to this ticket? Thanks!

oguzkocer commented 2 years ago

@hypest You're correct, I didn't receive the previous ping. We no longer use Jitpack as we started publishing our artifacts to our S3 Maven through a Buildkite job instead. I created a PR to update the integration instructions in README: #954

@Kodak1234 If you add the following maven url to your repositories, you should be able to fetch the latest artifacts:

repositories {
    maven { url "https://a8c-libs.s3.amazonaws.com/android" }
}

Please feel free to ping me if you have any issues.

Kodak1234 commented 2 years ago

I'm still unable to pull the latest artifacts.

allprojects {
    repositories {
        google()
        jcenter()
        maven { url "https://a8c-libs.s3.amazonaws.com/android" }
        maven { url 'https://jitpack.io' }
    }
}
oguzkocer commented 2 years ago

@Kodak1234 ~Which version are you trying to fetch?~ Please note that only the latest few versions are available in the S3 maven repository, v1.5.4 being one of them.

~If you can share the line where you're adding the dependency, that'd be helpful.~

Edit: The issue was most likely because of the change in artifact path which I've commented on separately below. Just making this edit, so it points to the correct comment.

oguzkocer commented 2 years ago

@Kodak1234 I forgot that we changed the path of the artifact. Can you update your dependency as below and try again?

dependencies {
    api "org.wordpress:aztec:v1.5.4"
}

Sorry about that confusion!

Kodak1234 commented 2 years ago

@oguzkocer Please can you guys write a documentation on the best way to implement this editor in recyclerview, using a simple textview does not render all the spans properly.

oguzkocer commented 2 years ago

@Kodak1234 I unfortunately don't have experience working with the library, but I'll bring it up internally to see if anyone who has experience have the bandwidth to write such a documentation.

In the meantime, your best best might be to look into how it's used in other WordPress projects. Here is the module it's used in the Gutenberg project and here is the one for WordPress-Android. In WordPress-Android, it might be used in other modules as well, so you might want to do a full search in the project.

I'll follow up about the internal discussion next week. Hope that helps!

hypest commented 2 years ago

Please can you guys write a documentation on the best way to implement this editor in recyclerview

👋 @Kodak1234, there doesn't seem to be available bandwidth from the main maintainers at the moment to dive into this request, sorry 😞 . That said, the links @oguzkocer shared above can be a good resource to check.

using a simple textview does not render all the spans properly.

I'd suggest to open a new ticket to focus on the specific usecase, ideally including links to code that can be reviewed. Aztec has been designed to be a replacement to an EditText, and shouldn't need any special handling to make it work in a RecyclerView. I could be forgetting something though so yeah, best if, in a separate ticket, you can provide a code example that fails to render the spans. Thanks!