Add KYC alert for all solutions and SSH alert

[samaradel]

Description

Describe the changes introduced by this PR and what does it affect

Changes

Add KYC and SSH keys alert for all solutions

Related Issues

• 🐞 [Bug]: Add the same initial two messages for sshkey & KYC for all solutions like VMs #4219

Tested Scenarios

• Check the KYC alert for all solutions
• Check the SSH alert for all solutions

Updated:

• Test if the messages still exist for the solution, ensuring the SSH key has not been deleted.
• Test that the SSH messages disappear in case multiple SSH and key message still exists.
• Test after kyc process is done, all messages disappear.

Documentation PR

For UI changes, Please provide the Documentation PR on info_grid

To consider

Preliminary Checks:

• Preliminary Checks
  • Does it completely address the issue linked?
  • What about edge cases?
  • Does it meet the specified acceptance criteria?
  • Are there any unintended side effects?
  • Does the PR adhere to the team's coding conventions, style guides, and best practices?
  • Does it integrate well with existing features?
  • Does it impact the overall performance of the application?
  • Are there any bottlenecks or slowdowns?
  • Has it been optimized for efficiency?
  • Has it been adequately tested with unit, integration, and end-to-end tests?
  • Are there any known defects or issues?
  • Is the code well-documented?
  • Are changes to documentation reflected in the code?

UI Checks:

• UI Checks
  • If a UI design is provided/ does it follow it?
  • Does every button work?
  • Is the data displayed logical? Is it what you expected?
  • Does every validation work?
  • Does this interface feel intuitive?
  • What about slow network? Offline?
  • What if the gridproxy/graphql/chain is failing?
  • What would a first time user have a hard time navigating here?

Code Quality Checks:

• Code Quality Checks
  • Code formatted/linted? Are there unused imports? .. etc
  • Is there redundant/repeated code?
  • Are there conditionals that are always true or always false?
  • Can we write this more concisely?
  • Can we reuse this code? If yes, where?
  • Will the changes be easy to maintain and update in the future?
  • Can this code become too complex to understand for other devs?
  • Can this code cause future integration problems?

Testing Checklist

• Does it Meet the specified acceptance criteria.
• Test if changes can affect any other functionality.
• Tested with unit, integration, and end-to-end tests.
• Tested the un-happy path and rollback scenarios.
• Documentation updated to meet the latest changes.
• Check automated tests working successfully with the changes.
• Can be covered by automated tests.

General Checklist

• Tests included
• Build pass
• Documentation
• Code format and docstring
• Screenshots/Video attached (needed for UI changes)

[amiraabouhadid]

please add sshkey alert to casper labs too

since it needs it

[samaradel]

@amiraabouhadid as per @AhmedHanafy725 any solution with a domain doesn't require sshkeys alert

[ramezsaeed]

Test scenarios:

• Test if the messages still exists for solution if the ssh key had been deleted.
• Test the ssh messages disappear i case multiple ssh and key message still exists.
• test after kyc process done all messages disappear.

[amiraabouhadid]

alert shows when i delete ssh keys

alert disappears after ssh key is generated

correct alerts shown for newly created account with no kyc and no ssh
FULL VM:

MICRO VM:

CAPROVER:

k8s:

algorand:

casperlabs:

and in every other solution in labs

[amiraabouhadid]

kyc alert disappears after kyc is verified correctly

and so does the ssh alert

[amiraabouhadid]

@AhmedHanafy725 just to confirm, kyc and ssh key alert should be on all solutions, correct?

[AhmedHanafy725]

@AhmedHanafy725 just to confirm, kyc and ssh key alert should be on all solutions, correct?

yes