apache/airflow

Parameterise key sorting in "Rendered Template" view

Open

#27,026 opened on Oct 13, 2022

View on GitHub
 (9 comments) (0 reactions) (0 assignees)Python (16,781 forks)batch import
area:webservergood first issuekind:feature

Repository metrics

Stars
 (44,809 stars)
PR merge metrics
 (Avg merge 7d 18h) (834 merged PRs in 30d)

Description

Description

Currently the https://github.com/apache/airflow/blob/main/airflow/www/views.py#L1254 has forceful sorting by keys. It'd be very nice to have this behaviour parameterised and not hardcoded

Use case/motivation

Sometimes we have big templates with few nested levels and it's very inconvenient to see higher level keys scattered across few screens located at the very bottom of template despite initially it was on the very start just because it was sorted by key forcefully.

Related issues

No response

Are you willing to submit a PR?

  • Yes I am willing to submit a PR!

Code of Conduct

Contributor guide

Research direction
Examine the views.py file around line 1254 where the forced sorting occurs. Look for the handler that renders the template and understand how the context is passed. Add an optional query parameter (e.g., 'sort keys') to the route, and modify the sorting logic to conditionally sort based on that parameter. Update the template if needed to allow users to toggle sorting.
Tech stack
python
Domain
backenddeveloper experience
Issue type
Feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
Half day
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
PythonFlask
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
65