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

Only send domainName #373

Merged
merged 1 commit into from
Oct 10, 2023
Merged

Only send domainName #373

merged 1 commit into from
Oct 10, 2023

Conversation

Janaka-Steph
Copy link
Contributor

@Janaka-Steph Janaka-Steph commented Oct 9, 2023

It closes #359

@what-the-diff
Copy link

what-the-diff bot commented Oct 9, 2023

PR Summary

  • Removal of Unused State Variables
    The developer team identified some variables (serverIP, email, and nodeName) within the code that were not being used. These have been safely eliminated to reduce unnecessary complexity.

  • Interface Simplification
    Following the removal of the aforementioned state variables, the corresponding input fields and their handlers in the user interface have also been removed. This simplifies the user experience as they now have fewer fields to complete.

  • Efficient Data Submission
    Our software's submit function, an aspect that handles data transmission, has been updated. It now only sends the domainName when communicating with our servers, which makes the data transmission leaner.

  • User Interface Layout Adjustment
    The layout for user-input elements has been rearranged. They got rid of a secondary column and optimized the placement of the remaining input fields. The goal is to make it easier for users to interact with the application.

@Janaka-Steph
Copy link
Contributor Author

@sekulicd Is backend ready for this?

@sekulicd
Copy link
Contributor

sekulicd commented Oct 9, 2023

@sekulicd Is backend ready for this?

yes

@tiero tiero merged commit 9f7cd40 into bit-gpt:main Oct 10, 2023
1 check passed
@Janaka-Steph Janaka-Steph deleted the only-domain branch October 13, 2023 12:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Domains: Improve submission UX
3 participants