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

10.2 - Changelog Test Results - Changed #35052

Closed
46 of 47 tasks
individual-it opened this issue Apr 18, 2019 · 10 comments
Closed
46 of 47 tasks

10.2 - Changelog Test Results - Changed #35052

individual-it opened this issue Apr 18, 2019 · 10 comments
Assignees
Labels
Milestone

Comments

@individual-it
Copy link
Member

individual-it commented Apr 18, 2019

Part of #35020

Changed

@phil-davis
Copy link
Contributor

Bump README.md doc links to 10.1 - #34403 - the link is invalid, issue #35035
It looks like it has been invalid at least since the docs conversion, and has been pointing into an odd place for a long time. PR #35036 to master.

@dpakach
Copy link
Contributor

dpakach commented Apr 24, 2019

Notify by email on group share can be done multiple times #35089

@patrickjahns
Copy link
Contributor

patrickjahns commented Apr 26, 2019

Tested tar_streamer - works as expected.
Do we have a webui/api test for this? Otherwise we might add downloading a tar archive to the acceptance tests ( one for zip and one for tar )

Note: request on a mac to test the tar

"GET /index.php/apps/files/ajax/download.php?dir=%2F&files=Photos&downloadStartSecret=xxxxx HTTP/1.1" 200 684106 "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/73.0.3683.103 Safari/537.36"

@individual-it
Copy link
Member Author

@patrickjahns thank you for testing, no tests for that AFAIK, please open an issue so we can schedule adding the tests

@individual-it
Copy link
Member Author

@dpakach please retest: Tidy up code for notification by email - it was fixed in RC2 #35137

@individual-it
Copy link
Member Author

updated the list above with changes in RC2 listed in e801e4e

@dpakach
Copy link
Contributor

dpakach commented May 6, 2019

@individual-it tested #35137 in RC2. Works fine.

@individual-it
Copy link
Member Author

new regression found while writing acceptance tests
Sharer gets "Email notification was sent!" message even when the email was not sent - #35218

@PVince81
Copy link
Contributor

close ?

please raise tickets for the other "to automate" tests before closing if not done already

@individual-it
Copy link
Member Author

I think we are done here for now
list for tests to be automated here #35207

@lock lock bot locked as resolved and limited conversation to collaborators May 20, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

5 participants