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

Elisa updating support #308

Merged
merged 3 commits into from
Feb 11, 2025
Merged

Elisa updating support #308

merged 3 commits into from
Feb 11, 2025

Conversation

Elisa-on-GitHub
Copy link
Collaborator

Quick fixes/updates on the Support page that remove references to services that no longer exist. We should have a better look at this page at some point to decide if its current form still fits our philosophy of the handbook.

Fixed Cartesius > Snellius.
Need more info about LISA and the data ingest service
Based on Peter's input:
- Deleted LISA (was replaced by Snellius)
- Changed reference to HPC Cloud to SURF Research Cloud and the topic we have on the handbook
@Elisa-on-GitHub Elisa-on-GitHub requested a review from a team as a code owner January 29, 2025 16:13
Copy link

netlify bot commented Jan 29, 2025

Deploy Preview for ubvu-open-handbook ready!

Name Link
🔨 Latest commit 4eaa841
🔍 Latest deploy log https://app.netlify.com/sites/ubvu-open-handbook/deploys/67a4893dca90bb0008b56e62
😎 Deploy Preview https://deploy-preview-308--ubvu-open-handbook.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 23
Accessibility: 82
Best Practices: 100
SEO: 90
PWA: -
View the detailed breakdown and full score reports

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

Copy link
Contributor

Summary

Status Count
🔍 Total 495
✅ Successful 378
⏳ Timeouts 1
🔀 Redirected 0
👻 Excluded 103
❓ Unknown 0
🚫 Errors 13

Errors per input

Errors in topics/bazis.qmd

Errors in topics/finding-existing-data.qmd

Errors in topics/data-documentation.qmd

Errors in guides/publish-and-share.qmd

Errors in topics/qualtrics.qmd

Errors in topics/software-licensing.qmd

Errors in topics/data-collection.qmd

Errors in topics/metadata.qmd

Errors in topics/data-publication.qmd

Errors in topics/snellius.qmd

Errors in topics/selecting-data.qmd

Copy link
Member

@chartgerink chartgerink left a comment

Choose a reason for hiding this comment

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

Looks good to me. However, I cannot judge whether the deletion of services makes sense, so would recommend for @Jolien-S or @peer35 or @TMHofstra to review this too 👍

Copy link
Collaborator

@Jolien-S Jolien-S left a comment

Choose a reason for hiding this comment

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

The current changes improve the page, so let's push this pull request, after a few minor changes I suggested in the text. I do see other things that need to be updated, so let's try to update the rest of the page as well.

Co-authored-by: Jolien-S <[email protected]>
Copy link
Contributor

github-actions bot commented Feb 6, 2025

Summary

Status Count
🔍 Total 29
✅ Successful 28
⏳ Timeouts 0
🔀 Redirected 0
👻 Excluded 1
❓ Unknown 0
🚫 Errors 0
Full Github Actions output

@Elisa-on-GitHub
Copy link
Collaborator Author

Elisa-on-GitHub commented Feb 6, 2025

@Jolien-S thank you! I made the requested changes

@chartgerink chartgerink merged commit f52ab5c into main Feb 11, 2025
5 checks passed
@chartgerink chartgerink deleted the Elisa-updating-support branch February 11, 2025 10:50
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.

None yet

3 participants