Skip to content

Changes static text to reflect it belongs to DeiC Storage #216

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

Open
wants to merge 3 commits into
base: edge
Choose a base branch
from

Conversation

aputtu
Copy link

@aputtu aputtu commented Mar 24, 2025

The aim has been to reflect DeiC Storage/DeiC better in the solution, so that screenshots can be made.
The static HTML snippets have been edited, so UCPH centric passages, sections, and functionality got altered or removed.

Examples are Seafile, Jupyter, and different login types for internal/external users. These are currently limited to UCPH.

  • As Dataverse is expected to integrate to DeiC Storage at a later phase; therefore mention on OID service have been kept, but commented out.
  • Skins in V2/V3 is not an option on DeiC Storage, so Quick tip regarding this have been removed.

Changes have sometimes been made to allow for later inclusion, simply by commenting out a HTML element.
In text passages text were removed, or altered.

There are still decisions left to be made regarding, say, external users, quotas, and so on. Once these decision have been made, it will be easier to decide which parts to completely exclude, reintroduce, and adjust.

aputtu added 3 commits March 20, 2025 08:53
Commented out quick tips services not in use, adjusted texts for active ones to leave out UCPH centric formulations.

Signed-off-by: Aputsiak Niels Janussen <aputtu@gmail.com>
Altering phrases with UCPH centric fomulations. 

Signed-off-by: Aputsiak Niels Janussen <aputtu@gmail.com>
Signed-off-by: Aputsiak Niels Janussen <aputtu@gmail.com>
@jonasbardino jonasbardino added the documentation Improvements or additions to documentation label Mar 24, 2025
extends to any trusted third parties at UCPH we work with to provide the Services,
for example University IT, which provides our UCPH OpenID login (again, only to
extends to any trusted third parties at DeiC Storage we work with to provide the Services,
for example University IT, which provides our WAYF login (again, only to
Copy link
Contributor

Choose a reason for hiding this comment

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

This should probably point to WAYF (https://deic.dk/en/WAYF) and refer to "our federated login relying on local organization user databases" or something like that.

<div class="tips-entry">
<h4 class="staticpage">YubiKey For Multifactor Authentication</h4>
<p>
Can I use a YubiKey security device for stronger and safer
authentication?
UCPH generally recommends securing your logins with MFA and offer
It generally recommends securing your logins with MFA and offer
Copy link
Contributor

Choose a reason for hiding this comment

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

It generally recommends -> Several organizations recommend
and offer -> and some offer

Selv i tilfælde, hvor nogen har opsnappet dit kodeord. Grundet generelle sikkerhedsovervejelser,
og nylig øgning af phishing-angreb, så anbefaler vi at slå 2FA til. Det sker ved at følge
en kort guide til
<a href="setup.py?topic=twofactor" target="_blank">opsætning af 2-faktor godkendelse</a>.
Copy link
Contributor

Choose a reason for hiding this comment

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

I think we may want to somehow refer to that fact that most organizations provide their own basic MFA solely integrated in the WAYF web login, but that users not coming through WAYF or just wanting MFA on SFTP and WebDAVS as well might gain from enabling this built-in extra factor.

@@ -162,7 +162,7 @@ <h4 class="staticpage">Digital Object Identifiers (DOIs) for your Data</h4>
<a href="/public/ucph-erda-user-guide.pdf">user guide</a> for more
Copy link
Contributor

@jonasbardino jonasbardino Mar 24, 2025

Choose a reason for hiding this comment

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

All references to anything ucph-erda here and in the above terms file need to go!
Depending on naming decisions all those links could e.g. be adjusted to /public/deic-storage-user-guide.pdf. Of course with some file like that actually included in the deployed state/wwwpublic/ folder in line with the integrators guide:
https://github.com/ucphhpc/migrid-sync/wiki/SiteCustomization

Copy link
Contributor

@jonasbardino jonasbardino 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, and my comments are not show-stoppers. We can merge now or leave the PR in place for further adjustments if you prefer?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants