Update main from dev #3 #913
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Created to address #798
Added proposed main headings
Added State of Open Con 24 talk link (Lise and Cody)
Paula Paul - Docs page for video recording links
Adding badge for try.a11y-theme-builder.finos.org
Removed some duplication of documentation, added section on feature requests
… theme to add a link to the header to launch our deployed Theme Builder demo in a new browser window
Update CONTRIBUTE.md
Live demo link
…ponents use of SelectProps to now use BaseSelectProps as mui changed the name in an earlier version, causing the build break
Fix dockerbuild error
#824: update dependency versions
…h 'people needing accommodations'
…sorders' replaced with 'people needing accommodations'
Grabbed a template from https://github.com/finos/software-project-blueprint/blob/main/NOTICE
Update NOTICE
Documentation Enhancement
fix #856 : Linter added for Code Cleanup
…-#889 #889: added introduction of Components
|
|
✅ Deploy Preview for dapper-ganache-45a60b ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for glistening-gecko-6b417a ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
|
/easycla |
this-commit-is-to-add-component-imagelist-issue#518
|
@TheJuanAndOnly99 emailed me and said that there is currently an error with the GitHub API that is reporting an EasyCLA error even though Kris fixed her email issues. He said that the CLA signature has been verified. Asking for reviews and then this PR should be good to merge. |
aaronreed708
requested review from
brycecurtis,
evangk6,
smithbk,
lwnoble and
PaulaPaul
|
Given the EasyCLA issue is resolved (or expected) I'm in favor of this merge - it's quite large but I think the risk of merge issues should be easy to 'fail forward' on- |
PaulaPaul
approved these changes
Jun 5, 2024
There was a problem hiding this comment.
Thanks @aaronreed708 - adding my approval here and happy to help with any post merge issues given the size of this batch.
Cheers!
lwnoble
approved these changes
Jun 6, 2024
|
Because this was ok'd by @TheJuanAndOnly99, merging. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Update main from dev to get benefit of dependency updates, documentation updates and new enhancements.