-
Notifications
You must be signed in to change notification settings - Fork 0
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
feat: merge ckanext-zarr as a subtree #160
Draft
ChasNelson1990
wants to merge
28
commits into
main
Choose a base branch
from
feat-merge-ckanext
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Conversation
This file contains 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
…#3) * _ * Adding helper functions / extended fjelltopp-theme layout * Adding helper functions / extended fjelltopp-theme layout * Adding helper functions / extended fjelltopp-theme layout * _ * scaled promoted bg image * _ * _ * Updated 'recently updated' * Moved package item template to fjelltopp theme * Removed a few helpers to clean code * Added the topics section * Updated : fjelltopp-theme added disclaimer about * Updated : Insights --------- Co-authored-by: toavina <[email protected]>
…run again (#4) Co-authored-by: toavina <[email protected]>
…eme without further changes (#5) Co-authored-by: toavina <[email protected]>
* Updates copyright value * Updates subscribe_to_updates.html subtext content * Dataset search fa-search hover fixed * Dataset search fa-search hover fixed * Removed the extended subcribe_to_updates.html * fix .fa-search color --------- Co-authored-by: toavina <[email protected]>
* Added a draft schema for Dublin Core and a dcat profile to include the elements in dcat export * Small change to doi label * Small changes for a few help_text * Updates to schema * Updates to schema * Updates to schema * Convert Json Schema to Yaml * Updates to UI * Updates to UI * Updates to schema * Added validator for date, a few fix for UI * Fixing multiple select * Macro for multiple select * Using yaml schema now * css fix * Small code fix * Default metadata for resources * Small fix for multiple_select_formatter * wip: add file uploader (#9) * Code improvements for additional_info.html and helpers.py * Remove excess of lines from zarr.css * Moved the namespace with others * Updated the language field as a single select * Updated the language field as a single select * Renamed schema * Commented profiles.py as WIP --------- Co-authored-by: toavina <[email protected]> Co-authored-by: Chas Nelson <[email protected]>
* About content updated with styling * About content updated with styling --------- Co-authored-by: toavina <[email protected]>
* feat: add a placeholder video to the homepage * fix: make iframe responsive * fix: make caption i18n friendly
* Removed insight section * Removed max-width for bottom search footer button * Updating search button * Updating search button * small comment * a bit of customization for datahub stats * fix: keep insights --------- Co-authored-by: toavina <[email protected]> Co-authored-by: Chas Nelson <[email protected]>
…ge (#22) * using groups with extra whose key is 'topic' and value is 'yes' * Removed filtering * Removed filtering * Updated the name 'topic' to 'Resource Type' --------- Co-authored-by: toavina <[email protected]>
* Removed extra 'Learn more about the repository' * Still under development banner color white / green background * update grays in datasets and resource read * changes inactive stage background color * needs rebase --------- Co-authored-by: toavina <[email protected]> Co-authored-by: Chas Nelson <[email protected]>
* feat: simplify schema * fix: minor improvement to subject placeholder
ChasNelson1990
requested review from
jonathansberry,
tomeksabala and
cooper667
September 1, 2024 11:21
I would be up for discussing this and what I was trying to do on Monday if people are free? |
Marking as not draft so Jenkins will build it |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Description
In order to reduce the number of commits, PRs and everything else - I have merged the git tree for ckanext-zarr into this repository.
The structure of the folder and all files remains the same.
I have moved the .github folder from the extension into the root of this project and updated it so that the Pipfile is used and tests are run on PRs in this repo.edit: I have been unable to get the GitHub Action tests working due to seeming incompatibilities with the ckan base image and pipenv.
Dependency Changes
Removes the submodule dependency.
Checklist