feature/#26_enhance_storage_stock_display #37

sst · 2025-01-31T23:34:20+01:00

sst commented

2025-01-31 23:34:20 +01:00

Got the stuff to work for me.
...did no excessive test of the overall function after changing this...

The performance did suffer a bit due to the recursions added for the number of parts annotations and the dynamic description foo...

...still feels good enough on my debug system - if you see an Issue I can try to profile and improve the performance again.
...But I value the new features more than the performance right now - if this becomes a problem in the future we still can disable it.

The search for storages is quite bad as there is no good way to search for the full path (except autogenerating a database field containing the path cyclically) - so the way to go is copy paste the uuid of the storage to move a stock to...

Got the stuff to work for me. ...did no excessive test of the overall function after changing this... The performance did suffer a bit due to the recursions added for the number of parts annotations and the dynamic description foo... ...still feels good enough on my debug system - if you see an Issue I can try to profile and improve the performance again. ...But I value the new features more than the performance right now - if this becomes a problem in the future we still can disable it. The search for storages is quite bad as there is no good way to search for the full path (except autogenerating a database field containing the path cyclically) - so the way to go is copy paste the uuid of the storage to move a stock to...

mhu was assigned by sst

2025-01-31 23:34:20 +01:00

sst added 14 commits 2025-01-31 23:34:20 +01:00

added option to expand sub-storage stocks to display all components from sub_storages 171b6b83f4

this comes in handy for assortment boxes using sub storages for each individual compartment which usually only holds a single stock

changed the prefix of stock uuid qr codes f6a878460d

fixed migrations 39b64aeb71

added form to change stock lot 3aa4225acb

showing stock uuid in stocks detail list 6eaef98c86

added link to template in stocks detail view e4e7456a5d

typos 3ec11cf092

added feature to relocate stocks to a different storage aefcc472ea

...had to deal with form prefixes at some places
...storage search does not work as expected yet :(

made the stock view more ugly by adding badges showing the number of stored parts, lots and substorages 6ae94e9ea4

fixed storage search 35b99e1b18

added template based dynamic description 6ca0ce483f

tweaked short parameter print e0c39d9d6e

added dynamic description to rest api 032c4fc838

added component dynamic description to stock rest api 462ed0c101

sst added spent time 2025-01-31 23:34:34 +01:00

10 hours

sst added 2 commits 2025-01-31 23:59:41 +01:00

rounded float values in short string, fixed displaying components without proper type adf152938d

improved display of dynamic description 871086c7b7

sst added 1 commit 2025-02-01 00:16:40 +01:00

added display of sub-storage when expanding sub-storages in stock view 6dc8f3bfef

mhu commented

2025-02-02 03:10:27 +01:00

Can you add your latest migrations to the repo?
I get a warning, that the models contain chainges that are not yet reflected in the migrations:
These are missing:

class Migration(migrations.Migration):

    dependencies = [
        ("parts", "0016_componentparametertype_interfix"),
    ]

    operations = [
        migrations.AlterField(
            model_name="componentparametertype",
            name="interfix",
            field=models.CharField(
                blank=True,
                help_text="char to be used as decimal point in dynamic description eg. 2R2",
                max_length=10,
            ),
        ),
        migrations.AlterField(
            model_name="componenttype",
            name="description_template",
            field=models.TextField(
                blank=True,
                help_text="Template to assemble the dynamic description. Use template syntax, access the component with 'object', parameters with 'param_*'.",
            ),
        ),
    ]

Other than that, it seems to work for me. The thing with the django templating for the component parameters is nice :) We need to document this somewhere. Probably make a help page on the app itslef to show some examples of the possibiliteis. i was only able to get it working by reading the source :)

Can you add your latest migrations to the repo? I get a warning, that the models contain chainges that are not yet reflected in the migrations: These are missing: ``` class Migration(migrations.Migration): dependencies = [ ("parts", "0016_componentparametertype_interfix"), ] operations = [ migrations.AlterField( model_name="componentparametertype", name="interfix", field=models.CharField( blank=True, help_text="char to be used as decimal point in dynamic description eg. 2R2", max_length=10, ), ), migrations.AlterField( model_name="componenttype", name="description_template", field=models.TextField( blank=True, help_text="Template to assemble the dynamic description. Use template syntax, access the component with 'object', parameters with 'param_*'.", ), ), ] ``` Other than that, it seems to work for me. The thing with the django templating for the component parameters is nice :) We need to document this somewhere. Probably make a help page on the app itslef to show some examples of the possibiliteis. i was only able to get it working by reading the source :)

sst added 1 commit 2025-02-02 11:52:58 +01:00

added migration adding the help text 1b48e8f283

sst commented

2025-02-02 11:56:40 +01:00

...added the missing migration.

...just the adding of help texts - did not expect this to be migrated to the db at all (it worked without migration as well)

...I would prefer adding a UI to configure the template including a live preview of a selected component.
...added it as a new ticket: #38

...added the missing migration. ...just the adding of help texts - did not expect this to be migrated to the db at all (it worked without migration as well) ...I would prefer adding a UI to configure the template including a live preview of a selected component. ...added it as a new ticket: #38

sst merged commit 8b3ef1af25 into develop

2025-02-03 22:20:05 +01:00

sst deleted branch feature/#26_enhance_storage_stock_display

2025-02-03 22:20:05 +01:00

sst referenced this issue from a commit

2025-02-03 22:20:07 +01:00

Merge pull request 'feature/#26_enhance_storage_stock_display' (#37) from feature/#26_enhance_storage_stock_display into develop

Sign in to join this conversation.

No reviewers