Skip to content

Added router public/index.php#24

Merged
weierophinney merged 1 commit into
laminas-api-tools:1.5.xfrom
djnotes:patch-1
Sep 16, 2020
Merged

Added router public/index.php#24
weierophinney merged 1 commit into
laminas-api-tools:1.5.xfrom
djnotes:patch-1

Conversation

@djnotes
Copy link
Copy Markdown
Contributor

@djnotes djnotes commented Aug 18, 2020

Seems like the public/index.php router is needed to run the project for development. So added the router to the composer serve command.

Q A

| Bugfix | yes

Description

Without passing the router script (index.php) on the composer command, the project does not load properly.

Copy link
Copy Markdown
Member

@michalbundyra michalbundyra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@djnotes thanks for your contribution.

As I remember we had issues with this line multiple times, and I believe it was the last version which suppose to work everywhere. Would you be able to provide more information, please?

  • php version
  • composer version
  • your OS

I've checked our other project and we have the same there, and I haven't seen any issue reports any time recently - see for example: https://github.com/mezzio/mezzio-skeleton/blob/3.6.x/composer.json#L107

Thanks!

@weierophinney weierophinney changed the base branch from master to 1.5.x September 11, 2020 15:36
Seems like the public/index.php router is needed to run the project for development. So added the router to the composer serve command.
@weierophinney
Copy link
Copy Markdown
Contributor

@michalbundyra I just verified that this fails on each of 7.1 - 7.4 for certain; I didn't bother trying 5.6 or 7.0, as doing so was going to require re-installing requirements (I'd originally created a project using 7.3). As such, I think we should merge. It's not a huge change, and we've had it before, so I think it falls under the "obvious fix" category.

@weierophinney weierophinney added this to the 1.5.3 milestone Sep 16, 2020
@weierophinney weierophinney added Bug Something isn't working and removed Awaiting Author Updates labels Sep 16, 2020
weierophinney added a commit that referenced this pull request Sep 16, 2020
Signed-off-by: Matthew Weier O'Phinney <[email protected]>
@weierophinney weierophinney merged commit 512578f into laminas-api-tools:1.5.x Sep 16, 2020
@michalbundyra
Copy link
Copy Markdown
Member

@weierophinney as it is failing here I guess it is also failing in mezzio, but... I haven't seen any reports there...

@weierophinney
Copy link
Copy Markdown
Contributor

@michalbundyra The difference here is that we have local assets associated with the skeleton, while the other skeletons do not.

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

Labels

Bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants