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

feat: optional editUrl config #75

Merged
merged 5 commits into from Jan 18, 2022
Merged

Conversation

thim81
Copy link
Contributor

@thim81 thim81 commented Jan 18, 2022

Motivation

The editUrl is handy for when the event catalog is linked to a public repository.
In some cases, the repository cannot be publicly available due to security policies.

This PR provides support for making the editUrl optional in the eventcatalog.config.js

Have you read the Contributing Guidelines on pull requests?

yes

@changeset-bot
Copy link

changeset-bot bot commented Jan 18, 2022

🦋 Changeset detected

Latest commit: 407d9e0

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@eventcatalog/core Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@netlify
Copy link

netlify bot commented Jan 18, 2022

✔️ Deploy Preview for eventcatalog-preview ready!

🔨 Explore the source changes: 407d9e0

🔍 Inspect the deploy log: https://app.netlify.com/sites/eventcatalog-preview/deploys/61e6cd8aeb25010008354c8d

😎 Browse the preview: https://deploy-preview-75--eventcatalog-preview.netlify.app

@thim81
Copy link
Contributor Author

thim81 commented Jan 18, 2022

Before screenshot
2022-01-18 at 12 11 31

After screenshot
2022-01-18 at 12 12 41

@thim81
Copy link
Contributor Author

thim81 commented Jan 18, 2022

To discuss:
With the "Edit this page" link disappearing at the bottom, the "last updated" shifts to the left.
Is this desired?

@thim81 thim81 changed the title Optional editUrl config feat: optional editUrl config Jan 18, 2022
@boyney123 boyney123 merged commit 4f4ef62 into boyney123:master Jan 18, 2022
@github-actions github-actions bot mentioned this pull request Jan 18, 2022
@thim81 thim81 deleted the hide-edit-link branch January 18, 2022 14:48
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

2 participants