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

Isolate client and refactor #91

Merged
merged 12 commits into from
Jan 8, 2021

Conversation

rboixaderg
Copy link
Contributor

@rboixaderg rboixaderg commented Jan 3, 2021

Refactor -- Use useLocation in all project
Refactor -- Delete unused variables in all project
Refactor -- Isolate client
Test -- Add data-test attribute in input field, to avoid break the test easily
Test -- Changed root user test, to default user test to avoid confusion with root user in guillotina
Test -- Delete it.only in content.js tests
Test -- Active autovacuum in guillotina e2e

#67

@rboixaderg rboixaderg marked this pull request as draft January 3, 2021 15:32
@rboixaderg rboixaderg marked this pull request as ready for review January 3, 2021 16:46
@jordic jordic merged commit 1c51173 into guillotinaweb:master Jan 8, 2021
@rboixaderg rboixaderg linked an issue Jan 9, 2021 that may be closed by this pull request
12 tasks
@rboixaderg rboixaderg removed a link to an issue Jan 9, 2021
12 tasks
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