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

update filter scrolltop #633

Merged
merged 1 commit into from
Feb 3, 2025
Merged

update filter scrolltop #633

merged 1 commit into from
Feb 3, 2025

Conversation

steveoni
Copy link
Collaborator

Pull request checklist

Please check if your PR fulfills the following requirements:

  • Make sure you are requesting to pull a topic/feature/bugfix branch (right side). Don't request your master!
  • Make sure you are making a pull request against the develop branch (left side). Also you should start your branch off our develop.
  • Check the commit's or even all commits' message styles matches our requested structure.
  • Check your code additions will fail neither code linting checks nor unit test.

Pull request type

Please check the type of change your PR introduces:

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

What is the current behavior?

Issue Number: N/A

What is the new behavior?

Does this introduce a breaking change?

  • Yes
  • No

Other information

Copy link
Collaborator

@luccasmmg luccasmmg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@luccasmmg luccasmmg merged commit 4da54a2 into dev Feb 3, 2025
7 checks passed
luccasmmg added a commit that referenced this pull request Feb 4, 2025
* [sitemap]: remove diallow all from sitemap

* Add harvest configs and supervisorctl

* Force rerun deployment

* Force rerun deployment

* empty

* Fix supervisord file permissions

* Add harvester to initial plugins

* Trigger CI

* Searchcard (#605)

* [sitemap]: remove diallow all from sitemap

* update searchcard style

* add aria-label to the footer

* update test

* update

* update dataset card

* Fix supervisord.conf path

* Fix supervisord file path in Dockerfile.dev

* Revert "Merge branch 'dev' of https://github.com/wri/wri-odp into dev" (#608)

This reverts commit 09e0761, reversing
changes made to c54ef01.

Co-authored-by: Luccas Mateus <[email protected]>

* Re apply multiple applications (#609)

* Revert "Merge branch 'dev' of https://github.com/wri/wri-odp into dev"

This reverts commit 09e0761, reversing
changes made to c54ef01.

* Reapply "Merge branch 'dev' of https://github.com/wri/wri-odp into dev"

This reverts commit 9399810.

---------

Co-authored-by: Luccas Mateus <[email protected]>

* Force rerun deployment

* Add log output after unit tests

* Move error logs in GitHub Actions

* Improve group package count api call (#611)

* [sitemap]: remove diallow all from sitemap

* empty

* improve groups package count fetch

* Move log print; Fix timeout

* update auto-url field in dataset form

* ODP-399 (#614)

* ODP-399

* Fix build

* scroll to the top on state change (#615)

* scroll to the top on stte change

* update pagination

* Swap back to Azure SSO from Okta SSO

* ODP-397 (#617)

* Fix requesteds by Nesma

* ODP-380 (#620)

* Allow CORS in tests

* Remove unnecessary build section from docker-compose.test

* Update docs

* update application and others (#624)

* update images and doc (#623)

* update userdoc search page

* Applications docs

* [ODP-400] Homepage search bar improvements (#612)

* update search field style

* update search field

* update color contrast

* update cta button (#627)

* Fix build

* User info (#628)

* Progress

* Only misiing csv export

* Almost finished

* Add docs

* Tests

* Fix test

* Orttho

* Rm poetry

* QA Fixes

* Fix ortto signup (Only when user accepts)

* Fixes onSuccess mutation download batch

* Downgrade git

* QA Fixes

* Delete test thats no longer relevant

* Fix test cost splitting

* Fix test cost splitting

* Rm console.log

* Change wording for button

* Marketing + Vulnerability

* fix issue with weird scrolling behaviour (#629)

* fix issue with weird scrolling bahaviour

* add height for mobile scroll

* Update to have timestamp

* Fix vulnerability

* [run_unit_tests.sh] Give verbose output

* fix search input alignment

* update search input fields

* Kim comments

* [main.yml] Temp. stop build logs

* Fix vulnerability

* [main.yml] Fix broken build

* [main.yml] Fix broken build

* [main.yml] Typo fix

* [main.yml] Fix image name

* Fix alignment recent added/updated

* Fix broken link

* ODP-397 Kim request

* update dataset edit url placeholder (#631)

* [docker-compose.test.yml] Temp. use redis 3

* [main.yml] Run build

* [main.yml] Disable build

* [docker-compose.test.yml] Add redis retry

* [docker-compose.test.yml] Dont run frontend

* [main.yml] Don't run frontend docker

* [docker-compose.test.yml] Remove redis health check

* [docker-compose.test.yml] Remove datapusher temporarily

* [docker-compose.test.yml] Fix redis container name

* [run_unit_tests.sh] Run traceback as long

* [run_unit_tests.sh] Update test command

* Small UI Fixes

* Fix font size

* [supervisor.harvest.conf] Temp. run harvester as root

* [main.yml] Run the build and push the images

* [docker-compose.test.yml] Revert changes

* [main.yml] Run tests

* [supervisor.*.conf] Use env variable to dynamically set user based on user running CKAN

* update filter scrolltop (#633)

* Kim requests(Download id, new fomratting date)

* Fix fonts

* Fix timezone

* Ortto forms

* Fix timezone

* Fix font size

* Fix typo

* Trigger CI

* Fix build

* Fix typo

---------

Co-authored-by: steveoni <[email protected]>
Co-authored-by: Michael Polidori <[email protected]>
Co-authored-by: Muhammad Ismail Shahzad <[email protected]>
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.

2 participants