-
Notifications
You must be signed in to change notification settings - Fork 108
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
Add documentation regarding Custom preview annotation usage 🤩 📜 #307
base: master
Are you sure you want to change the base?
Conversation
Her I have added documentation for using Custom preview annotation with Showkase. I have added use cases and KSP vs KAPT differences. Also added docs about how to use this with KAPT.
I am seeing slow builds:
This is how the annotation looks Is there any other setup needed to make ksp incremental with the update to this library? |
@polivmi1 I think it might be better to open an issue for this. I'm not 100% sure if this has anything to do with the custom preview annotation implementation, but are you using that through a fork or something? |
@oas004 I am using 1.0.0-beta18 . It isn't published on github releases, but is available in https://mvnrepository.com/artifact/com.airbnb.android/showkase and should contain the latest changes. That's also the version that breaks. |
I tried copying the annotation you made in the sample project and it seems to work as expected. Is the project you are using public? |
Unfortunately not. Next week I will try to set up a sample project to reproduce the error. |
@oas004 this is a sample where it is reproducible: https://github.com/polivmi1/ShowkaseIncrementalCompilation |
Thank you! I will check this out! :) |
@oas004 I have updated the sample and I am not seeing Showkase to apply any of the Preview parameters like: Is that supported on ksp? |
It should support the same preview type as single preview in older versions😊 |
Hi everyone, my code is like this
also, I have added to my grade
but I still have errors like this issue #357 can anyone help me? |
I think maybe if we can take the issues related to this feature in separate issues that would be cool. I don't think they are directly related to this PR. I have commented on your issue @yoviekaputra :) Will try to have a look at that. I will try to rebase this branch. Do you think we should merge this PR @vinaygaba ? This is only documentation changes for the multi preview stuff. I think it would be nice to get that in for people using that in their code base :) Wdyt? |
Here I have added documentation for using Custom preview annotation with Showkase. I have added use cases and KSP vs KAPT differences. Also added docs about how to use this with KAPT.
This should not be merged before the next release is out to avoid confusion.
I'm not entirely sure about the placement and how the content is structured. Is there anything else that is needed to be mentioned @vinaygaba? :)