Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Preparation for Database integration #210

Open
wants to merge 15 commits into
base: master
Choose a base branch
from

Conversation

schuetzepaul
Copy link
Contributor

@schuetzepaul schuetzepaul commented May 31, 2024

Preparation work for integration of the database

This integrates the registration of component IDs (via barcode scanner or manual entry) as well as the upload to a database into the assembly sequence.

The registration of component IDs can either be done in the toolbar ...
Screenshot from 2024-05-31 09-45-02

or in the assembly sequence:
Screenshot from 2024-05-31 09-44-27
Screenshot from 2024-05-31 09-44-50

When entering the IDs via the input dialogue, they are nevertheless registered in the toolbar.

The information can then be pushed to the database via the workflow (or the button in the toolbar):
Screenshot from 2024-05-31 09-47-54

The user is warned about missing information ...
Screenshot from 2024-05-31 09-47-38

... or can verify before uploading the information:
Screenshot from 2024-05-31 09-49-47

A function to actually upload the information still has to be added and can be varied from assembly center to assembly center.

On the way to this, a functionality was added in the background that makes it possible to abort individual actions, otherwise the user e.g. would have been informed that the information cannot be pushed to the DB, but either the action would be marked as "Done" and disabled or at the next click (due to the way signals are connected in Qt) the popups would appear twice.

This closes #202

Copy link

@anurnberg anurnberg left a comment

Choose a reason for hiding this comment

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

Thanks for implementing. Looks good to me. There are two changes to the assembly parameters unrelated to the database, but I don't mind having them in the same PR.

@schuetzepaul
Copy link
Contributor Author

Thanks for implementing. Looks good to me. There are two changes to the assembly parameters unrelated to the database, but I don't mind having them in the same PR.

Huh, indeed. They were meant for #208 and accidentally slipped into some other commit. Changed and fixed history 😬

@schuetzepaul
Copy link
Contributor Author

schuetzepaul commented May 31, 2024

Notes from today's meeting:

  • Add ID for glue
  • Add comment field, maybe at the popup asking whether to push to the DB

@schuetzepaul schuetzepaul self-assigned this May 31, 2024
@schuetzepaul schuetzepaul marked this pull request as draft May 31, 2024 13:27
@schuetzepaul
Copy link
Contributor Author

This now in addition includes ...

  • Glue ID to be added within the workflow
  • Comment field at the point of pushing to the database
    Screenshot from 2024-06-05 11-21-08
  • Pushing metrology information to the database (see screenshot below)
    Screenshot from 2024-06-05 11-51-50

@schuetzepaul schuetzepaul marked this pull request as ready for review June 17, 2024 09:32
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.

Feature suggestion: Database integration
2 participants