Skip to content

Make the url logged in field:create a clickable link #6216

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 4 commits into
base: 13.x
Choose a base branch
from
Open

Make the url logged in field:create a clickable link #6216

wants to merge 4 commits into from

Conversation

DieterHolvoet
Copy link
Contributor

Since we started using clickable links in the drush en command, I thought we could use it in drush field:create too.

@DieterHolvoet DieterHolvoet force-pushed the feature/field-create-link branch from 160e60d to 3c50857 Compare April 28, 2025 12:23
@DieterHolvoet
Copy link
Contributor Author

@weitzman

Failed asserting that ' [success] Successfully created field 'field_test3' on unish_article type with bundle 'alpha'
[success] Further customisation can be done through the edit form.
[ASCII](length: 162) contains "http://dev/admin/structure/unish_article_types/manage/alpha/fields/unish_article.alpha.field_test3" [ASCII](length: 98).

Looks like those kind of terminal links aren't part of the error output and I can't seem to figure out how to get it. Can we remove that line?

@weitzman
Copy link
Member

Yes I think we can adjust the expectation or remove it - whatever you think.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@weitzman weitzman Awaiting requested review from weitzman weitzman is a code owner

@greg-1-anderson greg-1-anderson Awaiting requested review from greg-1-anderson greg-1-anderson is a code owner

Assignees
No one assigned
Labels
area-field
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@DieterHolvoet @weitzman