Skip to content
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

fix "not a subscriber" diagnostic to reflect the actual error... #823

Closed
srirambv opened this issue Apr 21, 2023 · 8 comments
Closed

fix "not a subscriber" diagnostic to reflect the actual error... #823

srirambv opened this issue Apr 21, 2023 · 8 comments
Assignees
Labels
bug Something isn't working ready-for-qa web3

Comments

@srirambv
Copy link
Collaborator

Description

Show proper message when a user doesn't select any POAP/NFT when creating a Web3 Talk room

Steps

  1. Create a premium subscription for Talk
  2. Create a room
  3. Do not select required NFT/POAP
  4. Click on Start Web3 call
  5. Shows Sorry, you are not a subscriber message instead of showing the actual reason

Actual Result

Wrong.Error.Message.mp4

Expected Result

Should show the actual reason for not creating the room

Additional Information

cc: @mrose17

@mrose17 mrose17 changed the title Show proper message when a user doesn't select any POAP/NFT when creating a Web3 Talk room sometimes subscription status is lost (e.g., when i go the site it says i'm a subscriber, but when I try to create a room, it says "not a subscriber") Apr 21, 2023
@mrose17
Copy link
Member

mrose17 commented Apr 21, 2023

@srirambv - i think that this is a different bug, specifically that sometimes the browser seems to lose track of the fact that a user is a subscriber. i occasionally see the same error when I do select POAPs or NFTs.

i've changed the title to reflect this. if you disagree, please let me know so we can drill down a bit.

@mrose17 mrose17 added the bug Something isn't working label Apr 21, 2023
@mrose17
Copy link
Member

mrose17 commented Apr 21, 2023

update: it looks like Sri called it right, so there are a couple of things we can debug here!

@tackley
Copy link
Collaborator

tackley commented Apr 21, 2023

The actual error is being hidden by that message. I can see from this interaction that the correct subscription cookies were set, the subscription server returned the error:

body: {"message":"Unable to verify web3 authorizations; error id: 9937e4fd-2dbd-4ff9-b68e-5f8ff86c41a3: No ADs given","code":400}

But chose to display the message "Sorry, you are not a subscriber", presumably as a catch-all error message.

Screenshot 2023-04-21 at 16 50 23

@tackley tackley removed their assignment Apr 21, 2023
@mrose17
Copy link
Member

mrose17 commented Apr 21, 2023

so, two things:

first, instead of "Sorry, you are not a subscriber" we should display body.message

second, do we know why the subscription service says it's unable to verify?

@mrose17 mrose17 changed the title sometimes subscription status is lost (e.g., when i go the site it says i'm a subscriber, but when I try to create a room, it says "not a subscriber") fix "not a subscriber" diagnostic to reflect the actual error... Apr 24, 2023
johnhalbert pushed a commit that referenced this issue Apr 24, 2023
@johnhalbert
Copy link
Contributor

@mrose17 I've pushed a fix for this. The messages returned from subscriptions are not well formatted for display on the client. So, instead of doing that I've updated the client to validate the request and print the appropriate message when a 400 is returned.

@srirambv this is ready for re-testing.

@mrose17
Copy link
Member

mrose17 commented Apr 24, 2023

hmmm... OK, let me see what kind of errors i see. cf., #850

@srirambv
Copy link
Collaborator Author

This is fixed now. I see the error Please choose one or more POAPs for participants or Please choose one or more NFT collections for participants.

823.mp4

@johnhalbert @mrose17 with selecting just moderator POAP/NFT the call room gets created. Should the message read to either select the required POAP/NFT or moderator POAP/NFT ?

@mrose17
Copy link
Member

mrose17 commented Apr 25, 2023

Cf., #850 in other words, a room is not required…

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working ready-for-qa web3
Projects
None yet
Development

No branches or pull requests

4 participants