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

Nova 4 #37

Merged
merged 22 commits into from
Apr 19, 2022
Merged

Nova 4 #37

merged 22 commits into from
Apr 19, 2022

Conversation

simonhamp
Copy link
Owner

@simonhamp simonhamp commented Apr 16, 2022

Nova 4 support

I managed to get some time to have a look at Nova 4 and attempt to get this package running on it. The bad news is that it wasn't as straightforward as I'd hoped 😞 but the good news (🎉 ) is that I've managed to use it as an opportunity to rethink and somewhat massively upgrade the whole thing.

⚠️ This is a major upgrade containing a number of breaking changes - so if your application code is in any way relying on the behaviour of this package, please familiarise yourself with the changes and prepare to make any necessary adjustments

What's new

What's changed

  • Column matching has been inverted - instead of matching your DB field name to the column data from your uploaded file, you now choose from available columns to align to your DB fields, which feels more natural (the UI for this is simplified too)
  • Files are now stored in a storage/app/csv-import directory

What's fixed

@simonhamp simonhamp marked this pull request as ready for review April 18, 2022 12:49
@simonhamp simonhamp mentioned this pull request Apr 18, 2022
@simonhamp simonhamp merged commit 7d4511e into main Apr 19, 2022
@simonhamp simonhamp deleted the feature/nova-4 branch April 19, 2022 17:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant