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

removed file-upload deprecation #1227

Merged
merged 1 commit into from
Dec 10, 2015
Merged

removed file-upload deprecation #1227

merged 1 commit into from
Dec 10, 2015

Conversation

djvoa12
Copy link
Contributor

@djvoa12 djvoa12 commented Dec 8, 2015

Deprecation Warning:

  • Observing the files attr is deprecated, use filesDidChange instead.

@djvoa12
Copy link
Contributor Author

djvoa12 commented Dec 8, 2015

@jrjohnson need help verifying this code works. It let me upload a simple text file, but it returns server error when I click on the link to the uploaded file, which I assume it's for direct download.

@jrjohnson
Copy link
Member

I think problems testing are related to ilios/ilios#1190

@jrjohnson
Copy link
Member

This will require #1230 to be merged so it can be tested.

@jrjohnson jrjohnson self-assigned this Dec 9, 2015
@jrjohnson
Copy link
Member

👍 works! @homu r+

@homu
Copy link
Contributor

homu commented Dec 9, 2015

📌 Commit e7bda7c has been approved by jrjohnson

@homu
Copy link
Contributor

homu commented Dec 9, 2015

⌛ Testing commit e7bda7c with merge 56e8806...

homu added a commit that referenced this pull request Dec 9, 2015
removed file-upload deprecation

Deprecation Warning:
- Observing the `files` attr is deprecated, use `filesDidChange` instead.
@homu
Copy link
Contributor

homu commented Dec 10, 2015

☀️ Test successful - status

@homu homu merged commit e7bda7c into ilios:master Dec 10, 2015
stopfstedt pushed a commit to stopfstedt/frontend that referenced this pull request Mar 6, 2024
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.

3 participants