gristlabs/grist-core

Implement __dict__ for Record objects

Open

#932 geöffnet am 9. Apr. 2024

Auf GitHub ansehen
 (3 Kommentare) (0 Reaktionen) (1 zugewiesene Person)TypeScript (3.613 Stars) (168 Forks)batch import
good first issuegouv.frpython

Beschreibung

Hey there 👋🏼

I now know of the RECORD function (thanks to MichaelGrist from the Discord server who kindly pointed me to it), but as a python user (and bad documentation reader) I feel like it would just make sense to be able to call dict(rec) and get a dictionary representation of the record.

Happy to implement if I get your go-ahead 🙂

Contributor Guide

Tech Stack
pythontypescript
Domain
databackend
Issue Type
feature
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
under 1 hour
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
blocked
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
Python dict methodGrist Record class location
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70
Research-Richtung
Locate the Record class definition in the Grist Python codebase (likely in 'records.py'). Add a ` dict ` method that returns a dictionary of the record's fields. Ensure it works with the existing `dict()` built in. Check for existing tests for the Record class and add a test. Review the linked Discord conversation for any additional context or design decisions.