Skip to content

#597 simplify attendance updates for members - #607

Open
Theophile-Madet wants to merge 16 commits into
masterfrom
597-simplify-attendance-updates-for-members
Open

#597 simplify attendance updates for members#607
Theophile-Madet wants to merge 16 commits into
masterfrom
597-simplify-attendance-updates-for-members

Conversation

@Theophile-Madet

@Theophile-Madet Theophile-Madet commented Jan 2, 2025

Copy link
Copy Markdown
Collaborator

Here a PR for #597. It's a request from Jon and the Vorstand.

I didn't spend much time on the look and on the texts yet, I wanted to show the changes to Jon first to see if it fits the needs, but the logic can already be checked. If you have improvement ideas, let me know.

@Theophile-Madet Theophile-Madet self-assigned this Jan 2, 2025
@Theophile-Madet Theophile-Madet linked an issue Jan 2, 2025 that may be closed by this pull request
@Theophile-Madet
Theophile-Madet marked this pull request as ready for review January 6, 2025 17:06
href="{% url "shifts:attendance_cant_attend" attendance.id %}">
<span class="material-icons">event_busy</span>
{% translate "I can't attend this shift" %}
</a>

@kerstenkenan kerstenkenan Jan 6, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe a little few words for this button {% translate "Cannot attend" %}? Better readable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Simplify attendance updates for members

2 participants