Skip to content
This repository has been archived by the owner on Mar 27, 2022. It is now read-only.

CSV Import Redesign #519

Merged
merged 32 commits into from Aug 30, 2015
Merged

CSV Import Redesign #519

merged 32 commits into from Aug 30, 2015

Conversation

seven1m
Copy link
Owner

@seven1m seven1m commented Aug 26, 2015

Before:

screen shot 2015-08-25 at 10 20 29 pm

After:

screen shot 2015-08-25 at 10 55 49 pm

Finishes #417

`errors.full_messages` will now read attribute + ' ' + error, as is the
default. Use `errors.values` for errors without attribute prepended.
Conflicts:
	db/schema.rb
Conflicts:
	.rubocop.yml
	Gemfile
	app/assets/stylesheets/app/tables.css.scss
	app/controllers/application_controller.rb
	app/models/family.rb
	app/models/search.rb
	app/views/administration/checkin/groups/update.js.erb
	app/views/administration/dashboards/show.html.haml
	config/routes.rb
	db/schema.rb
	spec/factories/people.rb
Conflicts:
	app/models/setup.rb
@seven1m seven1m added the wip label Aug 26, 2015
@seven1m
Copy link
Owner Author

seven1m commented Aug 26, 2015

Remaining issues:

  • screen does not refresh while import is being executed
  • need to test a very large file
  • remove old people import code
  • test importing using legacy_id
  • i18n
  • ability to set visibility of new users from import screen

seven1m added a commit that referenced this pull request Aug 30, 2015
@seven1m seven1m merged commit 65ef11e into master Aug 30, 2015
@seven1m seven1m deleted the import branch August 30, 2015 01:51
This was referenced Sep 7, 2015
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.

None yet

1 participant