Skip to content

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.

@jonasbardino jonasbardino added the documentation Improvements or additions to documentation label Mar 24, 2025
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?

aputtu and others added 10 commits August 11, 2025 15:51
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>
polish slightly

Signed-off-by: Jonas Bardino <bardino@science.ku.dk>
polish slightly

Signed-off-by: Jonas Bardino <bardino@science.ku.dk>
polish slightly

Signed-off-by: Jonas Bardino <bardino@science.ku.dk>
Co-authored-by: Aputsiak Niels Janussen <aputtu@gmail.com>
Signed-off-by: Jonas Bardino <bardino@science.ku.dk>
Co-authored-by: Aputsiak Niels Janussen <aputtu@gmail.com>
Signed-off-by: Jonas Bardino <bardino@science.ku.dk>
Co-authored-by: Aputsiak Niels Janussen <aputtu@gmail.com>
Signed-off-by: Jonas Bardino <bardino@science.ku.dk>
Co-authored-by: Aputsiak Niels Janussen <aputtu@gmail.com>
Signed-off-by: Jonas Bardino <bardino@science.ku.dk>
jonasbardino added a commit that referenced this pull request Aug 11, 2025
Basically takes PR #216 changes and reapplies to our currently used
`next` branch, with a few PR discussion issues addressed, too.
Thanks for the suggestions and feedback @aputtu.
@jonasbardino
Copy link
Contributor

Closing as changes here were merged through PR #314 instead.

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