Skip to content
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.

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

Feature/workspace autocomplete by type #329

Merged
merged 4 commits into from
Apr 18, 2023

Conversation

amstilp
Copy link
Contributor

@amstilp amstilp commented Apr 18, 2023

  • Add new autocomplete views that autocomplete specific workspace types.
  • Add a new method BaseWorkspaceAdapter.get_autocomplete_queryset() that can be overridden by projects defining custom database types.

Closes #291

Add a new autocomplete view and url for autocompleting workspace names
by type. Add an adapter method to the BaseWorkspaceAdapter that can
be overridden by custom workspace adapters defined by the user. Add
tests.

Note that some of the tests are breaking, I think because the
test workspace adapter remains registered after some other test?
A couple tests in WorkspaceDeleteTest class registered a new adapter,
but the tearDown method did not reset the registry. This caused later
tests to break.
@codecov
Copy link

codecov bot commented Apr 18, 2023

Codecov Report

Merging #329 (d97e81d) into main (6010c3b) will increase coverage by 0.00%.
The diff coverage is 100.00%.

❗ Current head d97e81d differs from pull request most recent head 12bdfac. Consider uploading reports for the commit 12bdfac to get more accurate results

@@           Coverage Diff            @@
##             main     #329    +/-   ##
========================================
  Coverage   99.79%   99.80%            
========================================
  Files         112      112            
  Lines       20466    20566   +100     
========================================
+ Hits        20425    20525   +100     
  Misses         41       41            
Impacted Files Coverage Δ
anvil_consortium_manager/urls.py 100.00% <ø> (ø)
anvil_consortium_manager/adapters/workspace.py 100.00% <100.00%> (ø)
...nvil_consortium_manager/tests/test_app/adapters.py 100.00% <100.00%> (ø)
anvil_consortium_manager/tests/test_views.py 99.90% <100.00%> (+<0.01%) ⬆️
anvil_consortium_manager/views.py 99.62% <100.00%> (+<0.01%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@amstilp amstilp merged commit 85360af into main Apr 18, 2023
@amstilp amstilp deleted the feature/workspace-autocomplete-by-type branch April 18, 2023 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add autocomplete view for workspace types
1 participant