inveniosoftware / inveniosoftware/flask-menu

Bring back register_menu decorator

Open
#93 2 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
54
Forks
43
PR merge metrics
No merged PRs in 30d

Description

## Is your feature request related to a problem? Please describe.
I have used `register_menu` a lot in my code in lots of different files, but it got very abruptly removed when upgrading flask_menu.

## Describe the solution you'd like
I would like it to stay. If not possible I think the following steps should be taken.
- It should be deprecated for a while and show a warning when used.
- It should be removed from the documentation here: https://flask-menu.readthedocs.io/en/latest/
- An example should be added to the documentation that describing alternative ways to accomplish the same.
- It should be added to the changelog in the documentation above

## Describe alternatives you've considered
- Staying on version <2.0.0. (But then I'm stuck with an old version of flask)
- Reimplement it myself and monkey-patching this module (if possible)

## Additional context
I don't know why this was removed.

Contributor guide

Open the contributing guide

Research direction

Start by checking the current public API and the documentation at flask-menu.readthedocs.io to determine how register_menu was removed and what alternatives exist. The issue does not name source files or tests. Done would require either restoring register_menu with a deprecation path or documenting an agreed alternative, plus updating the documentation and changelog.

Written by the indexing model from the issue text.

Assessment

Tech stack
flask, python
Domain
backend
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
32/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.