Please note: this is for the 8.x-4.x branch and has a dependency of https://www.drupal.org/project/simplytest/issues/3063298
We have the basics of a React application that will serve as the main end-user form for simplytest.
We have set up an initial component, found here: https://git.drupalcode.org/project/simplytest/blob/8.x-4.x/themes/simply...
This needs to be refactored to just be the project version selector, not specific to Drupal core.
There needs to be a new component for the autocomplete widget, where a user types the name of the project and this triggers the population of the select widget. The select widget should be hidden until the user selects a project from the autocomplete widget.
The autocomplete widget should pull from a new endpoint created to pass a string and search for a matching project.
The "DrupalSelection" component should be refactored to just be a "ProjectSelection" component that contains both the autocomplete and the select component. https://git.drupalcode.org/project/simplytest/blob/8.x-4.x/themes/simply...
All of the logic to populate the select dropdown needs to be replaced with the new endpoint, found in this issue: https://www.drupal.org/project/simplytest/issues/3063298
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | interdiff.txt | 631 bytes | sumitmadan |
| #7 | pull-projects-and-versions-3063304-7.patch | 17.25 KB | sumitmadan |
| #4 | pull-projects-and-versions-3063304-4.patch | 17.25 KB | sumitmadan |
Comments
Comment #2
nerdsteinYou can see the new wireframes in this issue/comment, for reference: https://www.drupal.org/project/simplytest/issues/2946735#comment-12516993
And, the new comps in the same issue and this comment: https://www.drupal.org/project/simplytest/issues/2946735#comment-12609706
Comment #3
sumitmadan commentedComment #4
sumitmadan commentedI have updated the react application to pull the project information.
Comment #5
sumitmadan commentedComment #6
nerdsteinAwesome -- thank you for working on this. I have the following feedback:
`simlytest` - looks like this should be `simplytest`
Otherwise, this looks great. Thanks again.
Comment #7
sumitmadan commentedI have updated the patch. Please review now.
Comment #8
nerdsteinLooks good - merging. Thanks again for your help!
Comment #10
nerdstein