Skip to content

Commit

Permalink
fix: fixed states sync
Browse files Browse the repository at this point in the history
  • Loading branch information
Aradhya-Tripathi committed May 9, 2022
1 parent 35504f8 commit 7f37488
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion bench/app.py
Original file line number Diff line number Diff line change
Expand Up @@ -550,7 +550,7 @@ def install_app(
if os.path.exists(os.path.join(app_path, "package.json")):
bench.run("yarn install", cwd=app_path)

bench.apps.sync(app, required=resolution, branch=tag)
bench.apps.sync(app_name=app, required=resolution, branch=tag, app_dir=app_path)

if not skip_assets:
build_assets(bench_path=bench_path, app=app)
Expand Down
4 changes: 2 additions & 2 deletions bench/bench.py
Original file line number Diff line number Diff line change
Expand Up @@ -170,7 +170,7 @@ def set_states(self):
except FileNotFoundError:
self.states = {}

def update_apps_states(self, app_name: Union[str, None] = None, branch: Union[str, None] = None, required:List = []):
def update_apps_states(self, app_dir, app_name: Union[str, None] = None, branch: Union[str, None] = None, required:List = []):
if self.apps and not os.path.exists(self.states_path):
# idx according to apps listed in apps.txt (backwards compatibility)
# Keeping frappe as the first app.
Expand Down Expand Up @@ -203,7 +203,7 @@ def update_apps_states(self, app_name: Union[str, None] = None, branch: Union[st
if app_name and app_name not in self.states:
version = get_current_version(app_name, self.bench.name)

app_dir = os.path.join(self.apps_path, app_name)
app_dir = os.path.join(self.apps_path, app_dir)
if not branch:
branch = (
subprocess
Expand Down

0 comments on commit 7f37488

Please sign in to comment.