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

pkp/pkp-lib#7167 Fix duplicated user settings and remove unused fields #7464

Merged
merged 11 commits into from
Dec 8, 2021

Conversation

jonasraoni
Copy link
Contributor

No description provided.

@jonasraoni jonasraoni force-pushed the bugfix/master/7167-fix-user-settings branch from 6a6eb8b to 7ff4422 Compare November 21, 2021 19:48
@jonasraoni
Copy link
Contributor Author

jonasraoni commented Nov 22, 2021

@asmecher I've tried to decrease the running time of Travis with some settings, but didn't change that much
OJS: 2:45 > 2:25
OPS: 1:45 > 1:39
OMP: 3:38 > 3:26

Also, what about removing the stale branches in pkp-lib, ojs, etc?
image

@jonasraoni jonasraoni force-pushed the bugfix/master/7167-fix-user-settings branch from 065cff5 to ae899cd Compare November 23, 2021 08:01
@jonasraoni jonasraoni force-pushed the bugfix/master/7167-fix-user-settings branch from 1da8fed to 348258d Compare November 24, 2021 12:44
@jonasraoni jonasraoni force-pushed the bugfix/master/7167-fix-user-settings branch from 0119590 to dd9b08a Compare December 3, 2021 14:13
@jonasraoni
Copy link
Contributor Author

@asmecher I've updated the PR to address the index, the tests are passing yet, so feel free to take a look again 😁

@asmecher asmecher merged commit 46d1226 into pkp:main Dec 8, 2021
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.

2 participants