You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
For all user accounts, we need to populate photoUrl and pronunciationUrl from their profiles with valid URLs, depending on whether these accounts have uploaded profile photo or voice recording in Content BB. In the beginning we should do this only for dev, when we release 6.1 we should do the same on production. Please note that this does not affect user's privacy in any way, this just puts things in the right way.
@shurwit and I were discussing this issue and we are thinking that it would be best if the client handled these URL updates in account profiles. The general idea is that when a user starts the client, it should check whether the user's profile contains photoUrl and pronunciationUrl, and if these are missing and this user has uploaded the respective profile photo and voice recording to the Content BB, the client should make a request to the Core BB PUT /account/profile API to set these two fields.
We believe that this rolling update to account profiles will not be an issue because we do not plan to make profile photos or pronunciation audios visible in the directory by default. Since users will need to set their profile photo and pronunciation audio to public visibility manually, it will not matter that some accounts will not have these fields populated until those users open the app.
Please let me know your thoughts on this plan. Thanks!
For all user accounts, we need to populate photoUrl and pronunciationUrl from their profiles with valid URLs, depending on whether these accounts have uploaded profile photo or voice recording in Content BB. In the beginning we should do this only for dev, when we release 6.1 we should do the same on production. Please note that this does not affect user's privacy in any way, this just puts things in the right way.
Source: rokwire/illinois-app#4402
More: https://rokwire-workspace.slack.com/archives/C0325LZQE6N/p1732695230462189
CC: @pmarkhennessy, @petyos, @roberlander2.
The text was updated successfully, but these errors were encountered: