Skip to content

Attempt to add the browse page link to eLife navigation#3258

Merged
3mcd merged 1 commit intoknowledgefutures:masterfrom
elifesciences:master
Mar 11, 2025
Merged

Attempt to add the browse page link to eLife navigation#3258
3mcd merged 1 commit intoknowledgefutures:masterfrom
elifesciences:master

Conversation

@erkannt
Copy link
Contributor

@erkannt erkannt commented Mar 11, 2025

Issue(s) Resolved

We are currently rolling out a new URL link in the elifesciences.org site header.
https://github.com/elifesciences/issues/issues/9146

Concerns

We have not been able to test the change locally, as no-one of our team of four was able to to run an instance of PubPub or Storybook locally. #3257

@erkannt
Copy link
Contributor Author

erkannt commented Mar 11, 2025

@3mcd As noted above we have not been able to check if this breaks anything. We are happy to check it with your help or trust your judgement on when to merge this.

@3mcd
Copy link
Collaborator

3mcd commented Mar 11, 2025

@erkannt Sorry about that! I think this change is good to merge. I will check it out on our test environment before promoting the change to production!

@3mcd 3mcd merged commit 6d87aae into knowledgefutures:master Mar 11, 2025
@nlisgo
Copy link
Contributor

nlisgo commented Mar 12, 2025

Looks good on the test environment: https://elife-container.duqduq.org/about

When can we expect this to rolled out to production?

gabestein pushed a commit that referenced this pull request Apr 1, 2025
davidcmoulton added a commit to elifesciences/pubpub that referenced this pull request May 21, 2025
Attempt to add the browse page link to eLife navigation (knowledgefutures#3258)
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.

4 participants