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

Add ADR about design system change #890

Merged
merged 2 commits into from
Oct 31, 2024
Merged

Add ADR about design system change #890

merged 2 commits into from
Oct 31, 2024

Conversation

j08lue
Copy link
Contributor

@j08lue j08lue commented Mar 15, 2024

Closes #889

Description of Changes

Adds ADR about design system change

Notes & Questions About Changes

Based on content previously included in refactor ADR #875

Validation / Testing

https://github.com/NASA-IMPACT/veda-ui/blob/adr/design-system-change/docs/adr/003-design-system-change.md

Copy link

netlify bot commented Mar 15, 2024

Deploy Preview for veda-ui ready!

Name Link
🔨 Latest commit d7cc874
🔍 Latest deploy log https://app.netlify.com/sites/veda-ui/deploys/672296e2f3ca7400081ccdba
😎 Deploy Preview https://deploy-preview-890--veda-ui.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@j08lue j08lue requested a review from sandrahoang686 October 30, 2024 20:28
@j08lue
Copy link
Contributor Author

j08lue commented Oct 30, 2024

@sandrahoang686 @hanbyul-here @faustoperez - I think this ADR is not controversial and accurately documents the decision made. Should we push it over the finish line?

@j08lue
Copy link
Contributor Author

j08lue commented Oct 31, 2024

Think this is trivial, so I'll go ahead and merge it. Thanks @AliceR.

@j08lue j08lue merged commit ac43f28 into main Oct 31, 2024
9 checks passed
@j08lue j08lue deleted the adr/design-system-change branch October 31, 2024 08:40
@AliceR AliceR mentioned this pull request Nov 11, 2024
AliceR added a commit that referenced this pull request Nov 12, 2024
## 🎉 Features
* Card image/description independent of hero image/description by
@dzole0311 in #1244

## 🚀 Improvements
* Cookie consent code cleanup by @snmln in
#1199 , and @hanbyul-here in
#1240 , and @hanbyul-here in
#1241
* Add ADR about design system change by @j08lue in
#890
* Update condition to run playwright tests on release branches by
@dzole0311 in #1228
* Update STYLE_GUIDE.md by @AliceR in
#1227
* Fix lint configuration by @AliceR in
#1219
* Add tests for the AOI feature specification by @AliceR in
#1216
* Set data catalog filters to be closed by default by @vgeorge in
#1243
* Update tsconfig and make nav interfaces exposable for consumption by
@sandrahoang686 in #1223

## 🐛 Fixes
* Hotfix to hide the external link badge from cards by @dzole0311 in
#1231

## New Contributors
* @vgeorge made their first contribution in
#1243

**Full Changelog**:
v5.9.0...v5.10.0
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.

Document rationale and decision about design system replacement
3 participants