Skip to content
This repository was archived by the owner on Jul 27, 2025. It is now read-only.

Redirect upload step#1323

Merged
Shpigford merged 2 commits intomaybe-finance:mainfrom
enderahmetyurt:import-view-right-place
Oct 18, 2024
Merged

Redirect upload step#1323
Shpigford merged 2 commits intomaybe-finance:mainfrom
enderahmetyurt:import-view-right-place

Conversation

@enderahmetyurt
Copy link
Contributor

Fix #1322

@Shpigford
Copy link
Member

I agree with the bug, but we likely need a bit more nuance to where this redirects.

There's an uploaded? method on the Import model that checks for an uploaded file so maybe we need some sort of if/else or case statement to check for the current step and redirect to the appropriate one?

@enderahmetyurt
Copy link
Contributor Author

Good point @Shpigford. How does it look now?

@Shpigford
Copy link
Member

@enderahmetyurt Works for me! Thanks!

@Shpigford Shpigford merged commit 4a3685f into maybe-finance:main Oct 18, 2024
@enderahmetyurt enderahmetyurt deleted the import-view-right-place branch October 18, 2024 13:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: View Import doesn't work as expected if the import file doesn't exist

2 participants