loopback-explorer should pick up hideRemoteMethod called after the explorer was initialized

[bajtos]

// works, User.login is hidden
User.disableRemoteMethod('login', true);
app.use(explorer(app, { basePath: '/' }));

// does not work, User.login is shown in the UI
app.use(explorer(app, { basePath: '/' }));
User.disableRemoteMethod('login', true);

See also strongloop/loopback-component-explorer#22, #402 and #651

[supasate]

@bajtos Let me take this. (Anyway, I'm a beginner to this community. If I do something wrong, please advise.)

I'll send one PR to this repo to make Loopback emits 'remoteMethodDisabled' event and one PR to loopback-component-explorer to re-create the swagger object when this event fired.

[bajtos]

I'll send one PR to this repo to make Loopback emits 'remoteMethodDisabled' event and one PR to loopback-component-explorer to re-create the swagger object when this event fired.

Awesome, thank you for the contribution. I can see the first pull request was already landed :)

[bajtos]

Released in loopback@2.28.0 and loopback-component-explorer@2.5.0. Enjoy :)

[supasate]

@bajtos In loopback-component-explorer, should we change the loopback version in devDependency to be "^2.28.0" to make it more explicit about minimum requirement?

[bajtos]

In loopback-component-explorer, should we change the loopback version in devDependency to be "^2.28.0" to make it more explicit about minimum requirement?

That's a good idea, could you please contribute that change yourself?

I am closing this issue as resolved.

[supasate]

@bajtos Done. I made a PR at strongloop/loopback-component-explorer#159.