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

BOX-65 add search page #36

Merged
merged 15 commits into from
Jun 30, 2021
Merged

BOX-65 add search page #36

merged 15 commits into from
Jun 30, 2021

Conversation

OlegBrony
Copy link
Contributor

@OlegBrony OlegBrony commented Jun 25, 2021

little pr to close previous holes...
lock: yarn, package
fixed reflect

OlegBrony added 11 commits June 6, 2021 18:41
adds search ability with trigger on field change, with debounce

ISSUES CLOSED: #61
adds search ability with trigger on field change, with debounce
ISSUES CLOSED: #61
adds search ability with trigger on field change, with debounce
ISSUES CLOSED: #61
babel plugin
export

ISSUES CLOSED: #61
added package from suggestions in review

ISSUES CLOSED: #61
changing view for search page
changes view of users list
changes view of card list

ISSUES CLOSED: #65
changed base button to match design

ISSUES CLOSED: #65
history, reflect (has bugs)

ISSUES CLOSED: #65
# Conflicts:
#	src/app/server.tsx
#	src/entities/card/organisms/card-preview.tsx
#	src/features/search-bar/models/index.ts
#	src/features/search-bar/molecules/search.tsx
#	src/features/search-bar/organisms/search-bar.tsx
#	src/pages/home/index.tsx
#	src/pages/search/model.ts
#	src/pages/search/page.tsx
#	yarn.lock
BREAKING CHANGE:
missed reflect

ISSUES CLOSED: #65
@OlegBrony OlegBrony added bug Something isn't working and removed feature labels Jun 25, 2021
@OlegBrony OlegBrony requested review from azinit and sergeysova June 25, 2021 17:37
@OlegBrony OlegBrony changed the title Feat/65 add search page BOX-65 add search page Jun 25, 2021
.gitignore Outdated Show resolved Hide resolved
azinit
azinit previously approved these changes Jun 26, 2021
Copy link
Contributor

@azinit azinit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

В принципе выглядит правдеподобно

Но а как ты yarn починил? Теперь точно не поломает ничего?)

sergeysova
sergeysova previously approved these changes Jun 26, 2021
@sergeysova sergeysova enabled auto-merge (rebase) June 26, 2021 18:40
# Conflicts:
#	.gitignore
#	src/pages/search/page.tsx
#	yarn.lock
@OlegBrony OlegBrony dismissed stale reviews from sergeysova and azinit via ba9e97f June 28, 2021 13:42
risen228
risen228 previously approved these changes Jun 28, 2021
@risen228 risen228 requested a review from azinit June 28, 2021 13:55
yarn.lock Show resolved Hide resolved
@sergeysova sergeysova disabled auto-merge June 30, 2021 16:40
@sergeysova sergeysova merged commit baf0215 into master Jun 30, 2021
@sergeysova sergeysova deleted the feat/61-add-search-page branch June 30, 2021 16:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants