Skip to content

Auto populate IDOM component registry #24

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.

Sign up for GitHub

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

Merged
merged 23 commits into from
Nov 2, 2021
Merged

Auto populate IDOM component registry #24

Changes from 1 commit
Commits
Show all changes
23 commits
Select commit Hold shift + click to select a range
9b0f382
Auto populate IDOM registered components
Archmonger Oct 29, 2021
7bf6842
fix styling
Archmonger Oct 29, 2021
e6e28ea
remove noqa
Archmonger Oct 29, 2021
80e6bd8
use regex instead of render
Archmonger Oct 30, 2021
419f5c1
string cleanup
Archmonger Oct 30, 2021
f3dbd6d
more robust matching
Archmonger Oct 30, 2021
00b7323
remove unneeded escape slashes
Archmonger Oct 30, 2021
7cf9aab
move _register_component to utils
Archmonger Oct 30, 2021
3368eaa
use absolute django idom import path
Archmonger Oct 30, 2021
30ad813
remove unused import
Archmonger Oct 30, 2021
8b500cb
add logging to auto registration
Archmonger Oct 30, 2021
f2c6096
clean up docstring and comments
Archmonger Oct 30, 2021
7c84ef3
remove unneeded sys.modules check
Archmonger Oct 30, 2021
daabf62
avoid multiple component registrations
Archmonger Oct 30, 2021
afe643e
use in instead of get
Archmonger Oct 30, 2021
5b2e619
fix _get_components docstring
Archmonger Oct 31, 2021
bc1666e
add type hints
Archmonger Oct 31, 2021
fa46d50
fix appconfig name
Archmonger Nov 1, 2021
d030b86
add component regex tests
Archmonger Nov 1, 2021
b7ea800
move selenium tests to folder
Archmonger Nov 1, 2021
7a9ba2d
precompile component regex
Archmonger Nov 1, 2021
6142cce
use for loop for testing regex
Archmonger Nov 1, 2021
9da3d52
Merge remote-tracking branch 'upstream/main' into populate-idom-compo…
Archmonger Nov 1, 2021
File filter

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
use in instead of get
  • Loading branch information
@Archmonger
Archmonger committed Oct 30, 2021
commit afe643ed4d515b797ebb62800172a5fcf1ee49ba
2 changes: 1 addition & 1 deletion src/django_idom/utils.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@


def _register_component(full_component_name: str) -> None:
if IDOM_REGISTERED_COMPONENTS.get(full_component_name):
if full_component_name in IDOM_REGISTERED_COMPONENTS:
return

module_name, component_name = full_component_name.rsplit(".", 1)
Expand Down