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

Feature/field examples. Closes #322 #324

Merged
merged 3 commits into from
Sep 29, 2021

Conversation

james2doyle
Copy link
Contributor

Closes #322

On top of the examples being put in, I added a test for the field-details component

@james2doyle
Copy link
Contributor Author

hmm weird, not sure why a class is just completely missing in lumen

@shalvah
Copy link
Contributor

shalvah commented Sep 16, 2021

InteractsWithViews - Lumen doesn't come with views support by default. Also, was that class present in Laravel 6? That's where the test fails.

@james2doyle
Copy link
Contributor Author

Should I just remove the test then? Seems like its only applicable to half to app

@shalvah
Copy link
Contributor

shalvah commented Sep 27, 2021

Yeah, I think we can remove the test/

@shalvah shalvah merged commit e4d7e17 into knuckleswtf:master Sep 29, 2021
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.

Should the example params/body be used in the try-it-out fields?
2 participants