balderdashy / balderdashy/sails
Sails 1.0 + `sails-auth-new` plugin?
- Dominant language
- JavaScript
- Stars
- 22.8k
- Forks
- 1.9k
- PR merge metrics
- No merged PRs in 30d
Description
**Sails version**: 1.0
**Node version**: 8.10.0
**NPM version**: 5.0.4
**DB adapter name**: postgres
**DB adapter version**: N/A
**Operating system**: Windows10
Hi,
A lot of hooks used to add controllers using sails.controllers, since sails 1.0 that functionality is broken...
and a lot of useful dependencies do not work any more, I really am very disappointed in the sails team, so many useful dependencies stop working after each update, it is hard to work with this framework this way...
I tried using the sails-auth-new dependency and it doesn't work with sails 1.0 because it attempts to register its controllers which causes:
> warn: Ignored attempt to bind route (/logout) to unknown action { controller: 'AuthController',
> skipAssets: true,
> action: 'logout' }
I am going to fork this dependency and try and fix it,
how can i register a controller?
Contributor guide
Research direction
Start by reproducing the sails-auth-new dependency issue on Sails 1.0 and inspect its use of sails.controllers and the /logout route binding. Compare the controller registration behavior with the warning about the unknown AuthController.logout action; done means the dependency's controller and route work without that warning.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript, node.js
- Domain
- backend
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100