"Provided by" field in expander on data search is wrong#8
Merged
7 commits merged intoOct 8, 2010
Merged
Conversation
added 7 commits
October 1, 2010 12:57
…eoNode#774)" This reverts commit 1fc89b5.
This also updates the search results to use the GMD schema instead of CSW.
Member
|
David, thanks for extending the metadata integration and fixing a few of those silent template bugs. I like the refactor of the xml parsing functions and the profile links everywhere. I just have a question related to keywords (and the keyword_list method), are we supposed to allow keywords with more than one word, i.e. "World Bank", should we makke the keywords_list function smarter to not split by spaces inside quotes? In any case the patch looks in pretty good shape to be merged with master. |
Contributor
Author
|
There's lots of ways we could do the split for keywords (I personally think splitting on commas would be more convenient than quote marks, for example). I think we can safely defer that decision to a later release, however (probably 1.0.1). |
jj0hns0n
referenced
this pull request
in jj0hns0n/geonode
Jul 19, 2012
use style name instead of style object
Vampouille
pushed a commit
to Vampouille/geonode
that referenced
this pull request
Sep 12, 2017
update httplib2 and dockerfile so that it builds
goudine
pushed a commit
to goudine/geonode
that referenced
this pull request
Feb 1, 2018
enable karma tests running using node_modules
Codecov Report
@@ Coverage Diff @@
## master #8 +/- ##
=========================================
Coverage ? 41.16%
=========================================
Files ? 178
Lines ? 19565
Branches ? 3227
=========================================
Hits ? 8054
Misses ? 10893
Partials ? 618 |
mattiagiupponi
added a commit
that referenced
this pull request
May 28, 2026
# This is the 1st commit message: Replace flake8 and black with ruff # This is the commit message #2: Replace flake8 and black with ruff # This is the commit message #3: Update geonode/base/views.py Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com> # This is the commit message #4: Clear-text logging of sensitive information' Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com> # This is the commit message #5: 'CodeQL / Information exposure through an exception' Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com> # This is the commit message #6: Potential fix for pull request finding 'CodeQL / URL redirection from remote source' Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com> # This is the commit message #7: PR review suggestions # This is the commit message #8: PR review suggestions # This is the commit message #9: Potential fix for pull request finding 'CodeQL / Information exposure through an exception' Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com> # This is the commit message #10: fix PR review suggestion Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com> # This is the commit message #11: fix PR review suggestion Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com> # This is the commit message #12: Potential fix for pull request finding 'CodeQL / Information exposure through an exception' Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com> # This is the commit message #13: fix PR review suggestion Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com> # This is the commit message #14: [Fixes #14244] Do not return WWW-Authenticate inside API exception response (#14265) * Do not return WWW-Authenticate inside API exception response * Test WWW-Authenticate stripped * Fix test docstring # This is the commit message #15: [Fixes #14262] Avoid returning all permissions in dataset owner payload (#14263)
This pull request was closed.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Work done for http://projects.opengeo.org/CAPRA/ticket/774 . This branch includes a little extra review of the metadata stuff so it would be good to at least get review from Ariel.