Skip to content
This repository has been archived by the owner on Jul 14, 2022. It is now read-only.

Modal component refactor #391

Merged
merged 18 commits into from
Jul 10, 2019
Merged

Modal component refactor #391

merged 18 commits into from
Jul 10, 2019

Conversation

AlicjaSzu
Copy link
Contributor

@AlicjaSzu AlicjaSzu commented Jul 1, 2019 •
edited
Loading

I want to merge this change because...

Screenshots

Pull Request Checklist

  1. All visible strings are translated with proper context.
  2. All data-formatting is locale-aware (dates, numbers, and so on).
  3. The changes are tested.
  4. The code is documented (docstrings, project documentation).
  5. Changes are mentioned in the changelog.

@AlicjaSzu AlicjaSzu added the wip Work in progress label Jul 1, 2019
@AlicjaSzu AlicjaSzu self-assigned this Jul 1, 2019
@AlicjaSzu AlicjaSzu force-pushed the feature/modal-refactor branch 4 times, most recently from 8731cce to 8d7a6b9 Compare July 8, 2019 16:08
@AlicjaSzu AlicjaSzu added done and removed wip Work in progress labels Jul 9, 2019
@AlicjaSzu AlicjaSzu requested review from mateuszkula, jwm0 and bogdal July 9, 2019 11:55
@AlicjaSzu AlicjaSzu force-pushed the feature/modal-refactor branch from 43548b0 to 0843219 Compare July 9, 2019 14:31
@bogdal bogdal merged commit e67e3c4 into master Jul 10, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@jwm0 jwm0 jwm0 left review comments

@bogdal bogdal bogdal approved these changes

@mateuszkula mateuszkula mateuszkula approved these changes

Assignees

@AlicjaSzu AlicjaSzu

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@AlicjaSzu @bogdal @jwm0 @mateuszkula