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

🐞 [Bug]: Normalize the virtual machine subdomain #2564

Closed
1 task done
Mahmoud-Emad opened this issue Apr 15, 2024 · 8 comments
Closed
1 task done

🐞 [Bug]: Normalize the virtual machine subdomain #2564

Mahmoud-Emad opened this issue Apr 15, 2024 · 8 comments
Assignees
Labels
type_bug Something isn't working
Milestone

Comments

@Mahmoud-Emad
Copy link
Contributor

Is there an existing issue for this?

  • I have searched the existing issues

which package/s did you face the problem with?

Dashboard

What happened?

The subdomain is too long

Steps To Reproduce

  1. Go to the Dashboard
  2. Deploy a virtual machine
  3. Try to attach a subdomain to it

which network/s did you face the problem on?

Dev

version

2.3.4

Twin ID/s

143

Node ID/s

NA

Farm ID/s

NA

Contract ID/s

NA

Relevant screenshots/screen records

image

Relevant log output

NA
@Mahmoud-Emad Mahmoud-Emad added the type_bug Something isn't working label Apr 15, 2024
@sameh-farouk
Copy link
Member

Can users choose a subdomain without a prefix, if the name is not reserved? If not feasible, we can start with the user's chosen name and add a suffix afterward.

@xmonader xmonader added this to 3.14.x Apr 16, 2024
@xmonader xmonader added this to the 2.5.0 milestone Apr 16, 2024
@xmonader xmonader moved this to Accepted in 3.14.x Apr 16, 2024
@zaelgohary zaelgohary self-assigned this Apr 16, 2024
@zaelgohary zaelgohary moved this from Accepted to In Progress in 3.14.x Apr 16, 2024
@ramezsaeed
Copy link
Contributor

@sameh-farouk
I think the prefix is important even for the user himself; so he can relate this domain to his solution.
in case the user has multiples VMs with multiple domains; it will be easier to know which domain related to which VM with the prefix.

@zaelgohary
Copy link
Contributor

zaelgohary commented Apr 16, 2024

Work completed:

Removed twinId & solution name and edited subdomain char limit to 5 + prefix (VM name).

@zaelgohary zaelgohary moved this from In Progress to Pending Review in 3.14.x Apr 16, 2024
@zaelgohary zaelgohary moved this from Pending Review to Accepted in 3.14.x Apr 21, 2024
@zaelgohary zaelgohary moved this from Accepted to In Progress in 3.14.x Apr 29, 2024
@zaelgohary
Copy link
Contributor

WIP:

Reverted previous changes and currently working on combining both the prefix and the subdomain as a name and allow the user to edit both. However, a validation would be triggered to check if a name contract exists with the same name or not to avoid post-deployment errors.

@zaelgohary zaelgohary moved this from In Progress to Pending Review in 3.14.x Apr 30, 2024
@zaelgohary
Copy link
Contributor

Work completed:

Created a validation rule to check if a name contract with the same name exists on the chain or not. Edited the subdomain to include the prefix by default and changed the max limit to 15 instead of 35. Also, edited the subdomain tooltip and the name in the table accordingly.

@MohamedElmdary MohamedElmdary moved this from Pending Review to In Verification in 3.14.x May 8, 2024
@A-Harby
Copy link
Contributor

A-Harby commented May 19, 2024

After creating a domain, the same name validation isn't working.

Maybe you should randomize naming every time dialogue is reopened.
screen-capture (3).webm

@A-Harby A-Harby moved this from In Verification to Accepted in 3.14.x May 19, 2024
@zaelgohary zaelgohary moved this from Accepted to In Progress in 3.14.x May 20, 2024
@zaelgohary zaelgohary moved this from In Progress to Pending Review in 3.14.x May 20, 2024
@zaelgohary
Copy link
Contributor

Work Completed:

Validate subdomain if tab is changed, change tab condition to v-if instead of v-show so the validation condition to work.

@zaelgohary zaelgohary moved this from Pending Review to In Verification in 3.14.x May 21, 2024
@A-Harby
Copy link
Contributor

A-Harby commented Jun 3, 2024

Verified, Devnet 17d861b.
The domain name is now validated with a minimum of 4 characters and a maximum of 30 characters, as well as validation for the same domain.
And the name given to the domain is a combination of solution type, twin ID, solution name, and a random character.
For the domain input size, I reported it separately here: #2842.

image
image
image

TC2124 - Add gateway to a vm

@A-Harby A-Harby moved this from In Verification to Done in 3.14.x Jun 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type_bug Something isn't working
Projects
Status: Done
Development

No branches or pull requests

6 participants