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

Issues/import errors #98

Draft
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

alienp4nda
Copy link
Collaborator

Solves #42 fuelly imports

Currency was saved as currency name (currency code) when the backend code was expecting only currency code.

Added a notification for existing users with the currency currently set improperly.

I would like a dev branch to push to and an added task within the github workflow to push a new rc image to docker hub without setting the new release to the latest.

@AlfHou
Copy link
Owner

AlfHou commented Jan 31, 2024

We might be able to add a DB migration mapping the faulty currency name (currency code) to only currency code. That way user won't have to select the currency again.

I would like a dev branch to push to and an added task within the github workflow to push a new rc image to docker hub without setting the new release to the latest.

I think that sounds like a good idea. Do you have permissions to set it up or do you want me to do it?

@alienp4nda
Copy link
Collaborator Author

I believe I have permission to create the branch. I'll just have to figure out how to limit a step to only run when pushes happen on specific branches. This way pushes to dev create a rc image and not mark it lastest.

@alienp4nda alienp4nda changed the base branch from master to dev February 5, 2024 18: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.

2 participants