Skip to content
This repository has been archived by the owner on Apr 25, 2023. It is now read-only.

fix: 1st bug hunt of october #100

Merged
merged 5 commits into from Oct 12, 2021
Merged

fix: 1st bug hunt of october #100

merged 5 commits into from Oct 12, 2021

Conversation

KaWaite
Copy link
Member

@KaWaite KaWaite commented Oct 11, 2021

What I've done

  • Fix dataset modal bug where the selected dataset wasn't cleared on close (before adding)
  • Make Storytelling and Button widgets appear on creation (NOT only if user-made content)
  • Popper margin warning issue with Button component's menu (used margin on component, popper doesn't like it so using popper's offset param)
  • Fix both Dashboard's project creation and workspace creation modals' issue where form wouldn't clear after successful creation. Project creation's modal also wouldn't go away if you hit Enter to send form.
  • Fix add/delete member bug where the UI doesn't show newly added member or remove deleted member until refresh

What I haven't done

Can't test add member bug on local env since there is only one user

Which point I want you to review particularly

Let me know if the add member fix doesn't seem like it'll work/change anything! Since I can't test it would like other opinions. Anyways, pretty certain it won't break anything.

@netlify
Copy link

netlify bot commented Oct 11, 2021

❌ Deploy Preview for reearth-web failed.

🔨 Explore the source changes: ff9cc36

🔍 Inspect the deploy log: https://app.netlify.com/sites/reearth-web/deploys/6163f87083ed03000884ecd6

@codecov
Copy link

codecov bot commented Oct 11, 2021

Codecov Report

Merging #100 (ff9cc36) into main (50de1f2) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #100   +/-   ##
=======================================
  Coverage   53.43%   53.43%           
=======================================
  Files          48       48           
  Lines         889      889           
  Branches      117      117           
=======================================
  Hits          475      475           
  Misses        364      364           
  Partials       50       50           

@KaWaite KaWaite marked this pull request as ready for review October 11, 2021 08:53
Copy link
Member

@HideBa HideBa left a comment

Choose a reason for hiding this comment

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

Looks good to me. Just one question. Plz, have a look.

Copy link
Contributor

@issmail-basel issmail-basel left a comment

Choose a reason for hiding this comment

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

Just 2 simple comments, everything looks fine to me.

@KaWaite KaWaite merged commit 1b90322 into main Oct 12, 2021
@KaWaite KaWaite deleted the fix-bugs-10-11 branch October 12, 2021 08:35
keiya01 pushed a commit that referenced this pull request Apr 25, 2023
* Add infobox padding

* infobox and text block padding

* fix titles and descriptions

* update textBlock padding IDs

* add Japanese translation

Co-authored-by: basel.issmail <basel.issmail@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
3 participants