-
Notifications
You must be signed in to change notification settings - Fork 30
Additions of New Tags Permission to Android #353
Additions of New Tags Permission to Android #353
Conversation
v1.0.5: Daffy 🦆
Merge pull request razeware#279 from razeware/development
v1.0.7: Domestic 🦆
v1.0.8: Daisy 🦆
v1.0.9: Donald 🦆
v1.1.0 merge
Emitron v1.1.1 - Enabled portrait mode for Video Player
Emitron v1.2.0 - Open-🦆 Development
v1.3.0: Captain's 🪵
…' into EMA-352-Unable_to_access_content
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Not sure I follow it completely, but it seems ok to me at first glance. Thanks!
Its there a way to actually test it, with user accounts? @sammyd |
If you sign up for a new kodeco account, I can give you the new permissions and you can give it a try? |
Here is the new account, LMK when you have done it. Also where in particular should test it in the app? Username: @dev_testing |
@sammyd Not sure if you missed this, have the access been granted to the account? |
@KelvnPere So sorry for the delay—I've been out of the office this week. This account should now have the required access. |
No issues, that's totally fine. Can you provide me like a keyword search so i can fully test the app with that account. In other words, the videos that one with that access should have access to. Thanks |
@sammyd still waiting on you for the above. Thanks |
@KelvnPere sorry I completely missed this comment. That account should have access to all videos. |
@sammyd I have tested it from my end and it looks good. Also, i think the dev_testing in the collection page was showing some contents locked which i feel has to do with the correct permissions. When i tested it with my normal work kodeco account it showed correctly and fine. This is good from my end. |
Based on the title, there were some new permissions that were added which includes "stream-team-videos" and "stream-personal-videos" which was causing some users to not have access to some contents. Also, 2 tests were written to verify the permission checks during the login session.
Here is a link to the issue #352