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.

Sign up for GitHub

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

Jump to bottom

docs(ui): add note about libreoffice to readme #992

Merged
merged 5 commits into from
Sep 10, 2024
Merged

docs(ui): add note about libreoffice to readme #992

merged 5 commits into from
Sep 10, 2024

Conversation

andrewrisse
Copy link
Contributor

Small doc update for UI about libreoffice when running locally

@andrewrisse andrewrisse self-assigned this Sep 6, 2024
@netlify Netlify
Copy link

netlify bot commented Sep 6, 2024 •
edited
Loading

✅ Deploy Preview for leapfrogai-docs canceled.

Name Link
🔨 Latest commit c262120
🔍 Latest deploy log https://app.netlify.com/sites/leapfrogai-docs/deploys/66e099fff1b49000080a4e9a

@andrewrisse andrewrisse changed the title init chore(ui): add note about libreoffice to readme Sep 6, 2024
@andrewrisse andrewrisse marked this pull request as ready for review September 6, 2024 13:19
@andrewrisse andrewrisse requested a review from a team as a code owner September 6, 2024 13:19
justinthelaw
justinthelaw reviewed Sep 6, 2024
View reviewed changes
Copy link
Contributor

@justinthelaw justinthelaw left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please also add this to the quick_start guide within the website documentation. I added the needs-docs label to this PR.

EDIT: NVM, that is containerized. Slow morning!

OPTIONAL - add the ubuntu apt for installing libreoffice too!

@justinthelaw justinthelaw added needs-docs PR Label - Docs required prior to merge and removed needs-docs PR Label - Docs required prior to merge labels Sep 6, 2024
justinthelaw
justinthelaw previously approved these changes Sep 6, 2024
View reviewed changes
jalling97
jalling97 previously approved these changes Sep 6, 2024
View reviewed changes
@andrewrisse andrewrisse changed the title chore(ui): add note about libreoffice to readme docs(ui): add note about libreoffice to readme Sep 6, 2024
jalling97
jalling97 previously approved these changes Sep 6, 2024
View reviewed changes
justinthelaw
justinthelaw requested changes Sep 6, 2024
View reviewed changes
bundles/dev/cpu/uds-bundle.yaml Outdated Show resolved Hide resolved
src/leapfrogai_ui/README.md Outdated Show resolved Hide resolved
@andrewrisse andrewrisse merged commit 522778c into main Sep 10, 2024
19 checks passed
@andrewrisse andrewrisse deleted the ui-libreoffice-doc-update branch September 10, 2024 21:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@justinthelaw justinthelaw justinthelaw approved these changes

@CollectiveUnicorn CollectiveUnicorn CollectiveUnicorn left review comments

@jalling97 jalling97 jalling97 left review comments

Assignees

@andrewrisse andrewrisse

Labels
chore
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@andrewrisse @jalling97 @justinthelaw @CollectiveUnicorn