You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Right now, there are a few actions supported and those too are hard coded.
It'd be really awesome if the application can support actions based on text without manually mapping text to the actions. Manually specifying all the actions looks really really bad.
The text was updated successfully, but these errors were encountered:
These are hardcoded so that the code is easier to understand for freshers. This might be a first hand experience for a lot of them, so in my opinion it would be better if we just keep things to the basic level.
While this suggestion is good, I believe it would be better if we keep its scope to optional and to be taught only if time permits. If the functionality is to be added here, we should add it in a new branch that is for the extension of the current code after the students are able to reach the current level of the codebase.
The whole purpose was to make things pluggable. Remove hard coded actions. Remove hard coded strings. Keep a seperate constants.py for holding strings. Make an abstract class which other plugins will use.
Right now, there are a few actions supported and those too are hard coded.
It'd be really awesome if the application can support actions based on text without manually mapping text to the actions. Manually specifying all the actions looks really really bad.
The text was updated successfully, but these errors were encountered: