MR looks good. Merged to 2.0.x
Will create a new release soon.
Thank you all for your contributions.. :)
The changes looks good. Thank you all for your contributions.
Merged to 2.x. Will push a D10 compatible release.
Thank you!
Looks good. Merged to 2.x.
Thank you for the contributions :)
Thank you for all the efforts & comments on the issue.
Merged the changes to 2.x.
Will be releasing a new D10 compatible release.
Shriaas โ credited JayKandari โ .
Changes Look good to me. Thank you @riyas_nr. And congratulations on your first ever drupal.org commit ๐
Thanks @pooja_sharma for the changes.
And congratulatinos ๐ for your first drupal issue merge. :)
Merged to 2.x
Thanks for testing @Chandra
Marking this as fixed, because this is already fixed in https://www.drupal.org/project/login_popup/issues/3288375 ๐ Automated Drupal 10 compatibility fixes Fixed .
Merged to 2.x. Thank you!
JayKandari โ made their first commit to this issueโs fork.
Merged to 2.x.
Creating a drupal 10 compatible 2.1.0 release.
This change is released to <2.0.4
. Thus marking this as fixed.
Pushed a quick change regarding this.
To test,
* first goto manage-form of your content type.
* Change the dtuber field's widget settings
* set "Show additional fields in widget" -> checked
* create a new content
* upload the video
* input additional fields
* save
* verify the video got uploaded with the desired values.
This is an initial draft of the functionality, will refactor and push improved version of this in future. Needs review of this functionality.
Thanks!
Added functionality in TEST upload form to be able to select video privacy status.
Hi @Liliplanet,
Can you pls give more details around point no 4 - what kind of display options would be helpful?
can you add more details to a child issue dedicated to this point?
JayKandari โ created an issue.
JayKandari โ created an issue.
JayKandari โ created an issue.
Taking this... Will be creating sub-tickets for individual new features. So that smaller items can be tracked separately.
Please take a look at the right section "Child issues" for individual progress.
Thanks!
Added a new field in the TEST form to upload videos to a playlist in youtube.
For this feature to be enabled for a different field, will have to work in a separate ticket.
JayKandari โ created an issue.
Hi @Liliplanet,
Thanks for reporting the error. It would help if you can also share the steps you took to reproduce thsi error.
As for the error code, since it is 400, then it means something wrong information is being sent to the youtube servers, thus that error.
For me, If I am trying to upload a video by the Test uploader or via the field, both are suceeding.
Can you try deauthorising the youtube account or maybe use another youtube account for testing?
I guess the problem could also be around the application setup on google console. maybe try that?
Thanks!
Hi @chike, This is fixed in
https://www.drupal.org/project/dtuber/issues/3347618
๐
Drupal 10 compatibility
Fixed
issue.
Please take a latest pull of 2.0.x branch. I'll push a new release tag for this.
Merged. Thanks for your Contributions ๐
Merged. Thanks for your Contributions ๐
JayKandari โ created an issue.
JayKandari โ created an issue.
JayKandari โ created an issue.
Update:
I've fixed a change containing refreshing of access and refresh token. Also moved that data from config to state, because this will cause issues during every deployment. Thus.
ticket for ref -
https://www.drupal.org/project/dtuber/issues/3347609
๐
access_token & refresh_token shouldn't be tracked in config move to state
Fixed
@Liliplanet : can you pls test with latest 2.0.x dev release once see if that error is still reproducible?
Thanks!
JayKandari โ created an issue.
Hi @Lilian,
Thanks but I am unable to reproduce this error:
Dtuber Error : refresh token must be passed in or set as part of setAccessToken
The suggested code is already present in the module. I tried removing the refresh_token and access_token from configs and was not able to reproduce this exact error. Do you mind sharing all the steps to reproduce this?
Created a separate ticket for tracking this ->
https://www.drupal.org/project/dtuber/issues/3347604
๐
Dtuber Error : refresh token must be passed in or set as part of setAccessToken
Active
Or maybe you can try to deauthorize the authentication and reauthorise from this config page (under "Google Authentication" section) -> /admin/config/media/dtuber and then try again to create a new content with the video?
Thanks!
JayKandari โ created an issue.
Hi @Liliplanet,
I've just pushed a minor fix to the mentioned issue
TypeError: array_unshift(): Argument #1 ($array) must be of type array, string given in array_unshift() (line 253 of /home/mysite/public_html/core/modules/file/file.module)
I will be creating a new release with these changes 2.0.1
Please upgrade to the latest release.
Also, thank you for the suggestions. But I feel those can be tackled in separate issues. Currently the module is bare minimum and lacks many of the mentioned features. I would like to add those. Let us discuss this in a separate ticket?
As for this ticket marking it as Fixed.
Thank you !!
Please provide detailed steps to reproduce this. Pls reopen if required. Thanks!
I am unable to reproduce this issue. Can we get a deailed steps to reproduce this?
Pls feel free to reopen if needed.
Thanks!
Thanks! @Liliplanet for reporting this issue. I've pushed a small changes to 2.0.x. Could you pls retest the scenario with latest dev release of 2.0.x branch ?
If all goes well will create a new release containing this change.
Thanks!