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

not working #214

Closed
leetomlee123 opened this issue Apr 18, 2019 · 23 comments
Closed

not working #214

leetomlee123 opened this issue Apr 18, 2019 · 23 comments

Comments

@leetomlee123
Copy link

Issue summary

Briefly describe the issue you are experiencing (or the feature you want to see added). Tell us what you were trying to do and what happened instead.

Remember, an issue is not the place to ask questions. You can use Stack Overflow for that.

Your environment

  • Kong version = 1.1.1
  • Kong Dashboard version = 3.6.0 (Kong Dashboard versions before 3.0.0 aren't supported anymore)
  • Node version = #.#.#
  • Npm version = #.#.#

Issue description

image

Current behavior

Tell us what happens.
any action not working, i`m use docker

Expected behavior

Tell us what should happen instead.

Steps to reproduce

Tell us how to reproduce this issue.

Good to have

Any screenshots? Browsers' logs?

@spiroo
Copy link

spiroo commented Apr 28, 2019

me too!
image

@CHOMNANP
Copy link

CHOMNANP commented May 2, 2019

I'm having the same issue :D

@huangtao1
Copy link

I'm having the same issue...

1 similar comment
@YixuanZeng
Copy link

I'm having the same issue...

@KylinHe
Copy link

KylinHe commented May 9, 2019

I'm having the same issue... ovo~

@espider
Copy link

espider commented May 15, 2019

I'm having the same issue

1 similar comment
@jlcoding
Copy link

I'm having the same issue

@akjamie
Copy link

akjamie commented Jun 2, 2019

having same issue

1 similar comment
@yuleiqq
Copy link

yuleiqq commented Jun 3, 2019

having same issue

@davidback0420
Copy link

스크린샷 2019-06-16 오전 10 33 53

I'm having the same issue.

@TreeZhiyuan
Copy link

Does this project no longer maintained?

@towardsyoung
Copy link

Having same issue 1 years ago...

1 similar comment
@ygm521
Copy link

ygm521 commented Jun 27, 2019

Having same issue 1 years ago...

@guohailong2006
Copy link

哈哈哈哈,需要一个前端来fixed掉它

@m344739968
Copy link

WechatIMG88

why?

@yifengjin
Copy link

Having same issue

@hugbe8
Copy link

hugbe8 commented Jul 18, 2019

having same issue

@2030405043
Copy link

Having same issue

1 similar comment
@kuoshijia
Copy link

Having same issue

@unil19
Copy link

unil19 commented Oct 9, 2019

there is some config missed for kong-schemas.js
image

you can either fill up your config for your version such as 1.1, 1.2, 1.3 or make the 1.0 config used by default when your kong version is above 1.0;
I solve the problems with the latter way by change the getter like this;
var Schema = { get: function(version) { var major = semver.major(version); var minor = major==0?semver.minor(version):0; return schemas[major + '.' + minor]; } };

@illyaMs
Copy link
Contributor

illyaMs commented Oct 25, 2019

@unil19 yeah, seems like a bad version check in schema helper (I must confess my hands are dirty here 😄).

Should be OK to use your second solution, since kong has a stable (1.x) version and (hopefully) those guys do respect semver.

Could you please create a MR, so the owner of the repo may merge it and publish? Seems like the issue is quite popular, so that'll be a great contribution.

@linuxdevopscn
Copy link

Having same issue

@illyaMs
Copy link
Contributor

illyaMs commented May 13, 2020

There's a fix done by @yaohx submitted in #220
Some tests did not pass, but I guess it won't be a big deal to fix them if needed.
We're using this forked version in our company currently: yaohx@369b5df

One more note is that the option provided above by @unil19 seems to be more elegant, imho (so maybe this could also be sent as a PR).

Whatever option is preffered, I think we need @PGBI to intervene as he's the maintainer (and thus the only person that can merge these fixes to upstream and release/tag an updated docker image).

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

No branches or pull requests