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

UI Bug in Settings menu, In App Center tab #843

Closed
amirhmoradi opened this issue Jul 13, 2022 · 6 comments
Closed

UI Bug in Settings menu, In App Center tab #843

amirhmoradi opened this issue Jul 13, 2022 · 6 comments
Assignees
Labels
good first issue Good for newcomers help wanted Extra attention is needed

Comments

@amirhmoradi
Copy link

Describe the bug
When in settings page, if I switch from Development to Production (from top left menu), the content of the 'In App Center' tab, in the 'Security' section does not refresh.

To Reproduce
Steps to reproduce the behavior:

  1. In 'Development' environment, in 'Settings' menu, 'In App Center' tab, disable HMAC
  2. In 'Production' environment, in 'Settings' menu, 'In App Center' tab, enable HMAC
  3. Go to any other page (ex 'Notifications')
  4. Select 'Development' environment
  5. Go back to 'Settings' menu, 'In App Center' tab => HMAC is disabled
  6. Switch the environment to 'Production' (do not change page)
  7. See the error: HMAC shows as 'disabled' but should be enabled.
  8. In the same 'Production' environment, refresh the page or change tabs and come back, HMAC shows as enabled.

Expected behavior
Page contents shall be fully updated when we switch environments. There should be some kind of guarantee for that as it is supposed to behave as 2 distinct environments.

Screenshots
Screenshot 2022-07-13 at 21 41 03
Screenshot 2022-07-13 at 21 40 46

Details (please complete the following information):

Additional context
None

@scopsy scopsy changed the title Hosted Novu: UI Bug in Settings menu, In App Center tab UI Bug in Settings menu, In App Center tab Jul 17, 2022
@scopsy scopsy added good first issue Good for newcomers help wanted Extra attention is needed labels Jul 17, 2022
@muhammadawais-rogers
Copy link

assign this to me please. thanks

@scopsy
Copy link
Contributor

scopsy commented Jul 20, 2022

Thank you @muhammadawais-rogers, just did

@scopsy
Copy link
Contributor

scopsy commented Aug 1, 2022

@muhammadawais-rogers how are you? Do you need any help with this? Let me know if I can be of any help here

@aggmoulik
Copy link
Contributor

@scopsy Can I pick this up ?

@scopsy
Copy link
Contributor

scopsy commented Aug 5, 2022

Sure thing, @muhammadawais-rogers I will un-assign you now. If you did start to work on it please let me know 🙏

@aggmoulik
Copy link
Contributor

@scopsy Can you help me with setting up the different environment for development and production.

aggmoulik added a commit to aggmoulik/notifire that referenced this issue Aug 6, 2022
scopsy added a commit to aggmoulik/notifire that referenced this issue Aug 7, 2022
scopsy added a commit that referenced this issue Aug 7, 2022
fix: HMAC Switch when switching environment
@scopsy scopsy closed this as completed Sep 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

4 participants