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

LinkGenerator #3

Open
f3l1x opened this issue Nov 23, 2017 · 5 comments
Open

LinkGenerator #3

f3l1x opened this issue Nov 23, 2017 · 5 comments
Labels

Comments

@f3l1x
Copy link
Member

f3l1x commented Nov 23, 2017

  • According to given id create a link to API endpoint.
  • Test a link generation with local php development server.
  • Maybe use nette/routing for internal router, it has constructUrl.
@f3l1x f3l1x added the feature label Nov 23, 2017
@f3l1x f3l1x added this to the v0.2 milestone Nov 23, 2017
@f3l1x f3l1x modified the milestones: v0.2, v0.3 Jan 16, 2018
@f3l1x f3l1x modified the milestones: v0.3, v0.4 May 30, 2018
@mabar mabar modified the milestones: v0.4, v0.5 Jul 4, 2018
@mabar mabar modified the milestones: v0.5, v0.6 Oct 3, 2018
@mabar mabar self-assigned this Jan 24, 2019
@mabar mabar modified the milestones: v0.6, v0.5 Feb 24, 2019
@mabar mabar modified the milestones: v0.5, v0.6 May 17, 2019
@mabar mabar removed this from the v0.6 milestone Jul 24, 2019
@TonyVlcek
Copy link
Contributor

Is this still somewhere in the road-map? It would be a really cool piece of functionality to have.

@mabar
Copy link
Contributor

mabar commented Aug 20, 2019

Probably in next release. There is still some work we need to do before link generator completion

@TonyVlcek
Copy link
Contributor

Nice, that sounds promising! :)

Not even sure that I would be able to help here, but just in case: Is this something worth looking into, or is #94 nearly there and any other MR would cause more harm than good?

@mabar
Copy link
Contributor

mabar commented Aug 20, 2019

Optional parameter default values, placeholders, base uri, classes mapping, ... we need to discuss these things internally first, sorry

@TonyVlcek
Copy link
Contributor

That makes sense. Thanks for the time you're putting into this and feel free to reach out if there's something less complex to be implemented.

@mabar mabar removed their assignment Nov 3, 2020
f3l1x pushed a commit that referenced this issue Nov 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging a pull request may close this issue.

3 participants