-
Notifications
You must be signed in to change notification settings - Fork 316
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
v1.1.1 #174
v1.1.1 #174
Conversation
- docker python install changes. pin watchdog version, turn off unneeded caching. - docker entrypoint changes. run cite process immediately, ensure no simultaneous runs. - first time setup fixes. don't parse config.yaml as yaml, instead customize with string replacement to preserve key order and comments. - add title and link fallbacks to citation component - fix start/end markers for blog post excerpts, and remove special characters from hidden search attribute - change member page from col layout to float - add multi-line option to regex_scan liquid plugin (to support start/end marker bug above) - code, portrait, post excerpt css - fix testbed image links
- fix card component css
- fix versioning workflow to correctly parse and split changelog
- (re)include user pr template
✅ Deploy Preview for lab-website-template ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work! I left a few minor comments and suggestions. Overall I thought it looked good!
An additional note:
- In the deploy preview I noticed on the homepage, under "Our Projects" the link for "browse our projects" results in a page not found.
Thanks for the review @d33bs , please take a look at my notes and recent commits. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems like @d33bs has done a pretty thorough review already. I made one comment, but otherwise it looks good to me.
See changelog for high level, user-facing changes.