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

Local fix for #148 #180

Merged
merged 2 commits into from
Feb 12, 2025

Conversation

eliotb
Copy link
Contributor

@eliotb eliotb commented Feb 11, 2025

Description

Localised fix for #148 (avoids a traceback). Another issue suggests ingkey should be refactored out of existence,
but that is a much bigger job.

Also fixes a traceback when loading shopping editor plugin (don't know an issue number for this one)

How Has This Been Tested?

Run gourmand, test editing of a recipe ingredient.

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Ignore unused arguments to update_ingredient_row (ingkey) GourmandRecipeManager#148

Signed-off-by: Eliot Blennerhassett <[email protected]>
GtkEntryCompletion should use set_text_column (doesn't have set_entry_text_column,
even when spelled correctly)

Signed-off-by: Eliot Blennerhassett <[email protected]>
Copy link
Contributor

@FriedrichFroebel FriedrichFroebel left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks!

@FriedrichFroebel FriedrichFroebel merged commit 4e3fda0 into GourmandRecipeManager:main Feb 12, 2025
14 checks passed
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.

2 participants