@max_morais_dmm i think u r right …is better to have it as a separate app …
btw i’ve got an error while installing …
bench get-app title_links https://github.com/mxmo-co/title_links
INFO:bench.app:getting app title_links
Cloning into 'title_links'...
remote: Counting objects: 45, done.
remote: Compressing objects: 100% (18/18), done.
remote: Total 45 (delta 12), reused 45 (delta 12), pack-reused 0
Unpacking objects: 100% (45/45), done.
Checking connectivity... done.
installing title_links
INFO:bench.app:installing title_links
Traceback (most recent call last):
File "/usr/lib/python2.7/runpy.py", line 162, in _run_module_as_main
"__main__", fname, loader, pkg_name)
File "/usr/lib/python2.7/runpy.py", line 72, in _run_code
exec code in run_globals
File "/home/frappe/frappe-bench/apps/frappe/frappe/utils/bench_helper.py", line 79, in <module>
main()
File "/home/frappe/frappe-bench/apps/frappe/frappe/utils/bench_helper.py", line 12, in main
commands = get_app_groups()
File "/home/frappe/frappe-bench/apps/frappe/frappe/utils/bench_helper.py", line 20, in get_app_groups
for app in get_apps():
File "/home/frappe/frappe-bench/apps/frappe/frappe/utils/bench_helper.py", line 76, in get_apps
return frappe.get_all_apps(with_internal_apps=False, sites_path='.')
File "/home/frappe/frappe-bench/apps/frappe/frappe/__init__.py", line 691, in get_all_apps
apps = get_file_items(os.path.join(sites_path, "apps.txt"), raise_not_found=True)
File "/home/frappe/frappe-bench/apps/frappe/frappe/__init__.py", line 820, in get_file_items
content = read_file(path, raise_not_found=raise_not_found)
File "/home/frappe/frappe-bench/apps/frappe/frappe/__init__.py", line 835, in read_file
from frappe.utils import cstr
ImportError: cannot import name cstr
Traceback (most recent call last):
File "/usr/local/bin/bench", line 9, in <module>
load_entry_point('bench', 'console_scripts', 'bench')()
File "/home/frappe/bench-repo/bench/cli.py", line 40, in cli
bench_command()
File "/usr/local/lib/python2.7/dist-packages/click/core.py", line 716, in __call__
return self.main(*args, **kwargs)
File "/usr/local/lib/python2.7/dist-packages/click/core.py", line 696, in main
rv = self.invoke(ctx)
File "/usr/local/lib/python2.7/dist-packages/click/core.py", line 1060, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "/usr/local/lib/python2.7/dist-packages/click/core.py", line 889, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/usr/local/lib/python2.7/dist-packages/click/core.py", line 534, in invoke
return callback(*args, **kwargs)
File "/home/frappe/bench-repo/bench/commands/make.py", line 28, in get_app
get_app(name, git_url, branch=branch)
File "/home/frappe/bench-repo/bench/app.py", line 60, in get_app
build_assets(bench_path=bench_path)
File "/home/frappe/bench-repo/bench/utils.py", line 144, in build_assets
run_frappe_cmd('build', bench_path=bench_path)
File "/home/frappe/bench-repo/bench/utils.py", line 499, in run_frappe_cmd
raise CommandFailedError(args)
bench.utils.CommandFailedError: ('build',)
Another small thing, looking at your code it call subscription app …probably is better to give a “definitive” name