- Issue created by @Kristen Pol
- Assigned to imalabya
- Status changed to Needs review
3 months ago 7:03am 15 August 2024 - 🇮🇳India imalabya Bangalore
@Kristen Pol Below are the findings
- Reviewed the SDDS SDC conversion guide
- Used the conversion script to generate the yml file and README.md
- Theiframe.component.yml
&README.md
file were generated inside demo_design_system/components/01-atoms/iframe
- Copied the demo_design_system/components/01-atoms/ideam to demo_design_system/starshot_demo/components/01-atoms/iframe
- Verified the SDC component in the SDC Registry - Issue was unassigned.
- Status changed to Needs work
3 months ago 6:07am 20 August 2024 - 🇺🇸United States Kristen Pol Santa Cruz, CA, USA
Thanks!
Note: The iframe component doesn't need to be added to the starshot_demo subtheme.
Also, can you compare your work against the recent dev changes? There was recently a commit of all baseline yaml and readme files. It would be helpful to compare your results against that to see if they align or the files should be updated with some of your suggestions
- Status changed to Needs review
3 months ago 11:21am 20 August 2024 - 🇮🇳India imalabya Bangalore
@kristen pol Removed the component from the starshot_demo subtheme.
The recent dev changes have made the
vertical_spacing
the required prop in the component.yml for iframe. IMO, it should be theurl
Updated the MR with the same. - Status changed to RTBC
3 months ago 6:05am 21 August 2024 - 🇺🇸United States Kristen Pol Santa Cruz, CA, USA
Agreed! I'll get this merged in, thanks.
- Status changed to Fixed
3 months ago 6:08am 21 August 2024 - 🇺🇸United States Kristen Pol Santa Cruz, CA, USA
Not sure why pipeline is being weird so merged it on the command line. Thanks!
Automatically closed - issue fixed for 2 weeks with no activity.