Skip to content

feat/AB#80711_remove_unnecessary_sort_possibilities_and_fix_name_sort #2126

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

Conversation

RenzoPrats
Copy link
Contributor

@RenzoPrats RenzoPrats commented Dec 4, 2023

Description

Feat: removed unnecessary sort options and fixed name sort in reference data.

Useful links

ticket
backend

Type of change

  • Improvement (refactor or addition to existing functionality)

How Has This Been Tested?

It has been tested accessing reference data verifying if now the only option to sort is by name and if it's working.

Screenshots

Peek 04-12-2023 13-49

Checklist:

( * == Mandatory )

  • * I have set myself as assignee of the pull request
  • * My code follows the style guidelines of this project
  • * Linting does not generate new warnings
  • * I have performed a self-review of my own code
  • * I have put the ticket for review, adding the oort-frontend team to the list of reviewers
  • * I have commented my code, particularly in hard-to-understand areas
  • * I have put JSDoc comment in all required places
  • * My changes generate no new warnings
  • * I have included screenshots describing my changes if relevant
  • * I have selected labels in the Pull Request, according to the changes with code brings

@RenzoPrats RenzoPrats added the enhancement New feature or request label Dec 4, 2023
@RenzoPrats RenzoPrats requested a review from a team December 4, 2023 17:12
@RenzoPrats RenzoPrats self-assigned this Dec 4, 2023
@RenzoPrats RenzoPrats marked this pull request as ready for review December 4, 2023 17:46
@RenzoPrats RenzoPrats changed the title removed unnecessary sort options and fixed name sort in reference data feat/AB#80711_remove_unnecessary_sort_possibilities_and_fix_name_sort Dec 5, 2023
@LenaelleL LenaelleL self-requested a review December 5, 2023 10:20
Copy link
Contributor

@unai-reliefapp unai-reliefapp left a comment

Choose a reason for hiding this comment

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

Hi @RenzoPrats , can you clean up the reference datas query in the backend please?
https://github.com/ReliefApplications/ems-backend/blob/beta/src/schema/query/referenceDatas.query.ts#L53-L68
The are some sort fields there(like created at) and some logic that can be erased as we only do it by name field now 👍
Thanks

@LenaelleL LenaelleL removed their request for review December 5, 2023 11:13
@unai-reliefapp unai-reliefapp removed the request for review from a team December 5, 2023 13:13
…ilities_and_fix_name_sort # Please enter a commit message to explain why this merge is necessary, # especially if it merges an updated upstream into a topic branch. # # Lines starting with '#' will be ignored, and an empty message aborts # the commit.
@AntoineRelief AntoineRelief merged commit 1ae1e27 into beta Dec 5, 2023
@AntoineRelief AntoineRelief deleted the feat/AB#80711_remove_unnecessary_sort_possibilities_and_fix_name_sort branch December 5, 2023 14:13
AntoineRelief pushed a commit that referenced this pull request Dec 5, 2023
….13...v2.2.0-beta.14) (2023-12-05) ### Bug Fixes CHANGELOG CHANGELOG.md CI LICENSE README.md TODO.md apps assets error_pages jest-shim.ts jest.config.ts jest.preset.js libs makefile migration-storybook.log node_modules nx.json package-lock.json package.json release.config.js tailwind.config.js tools tsconfig.base.json Changing form locale duplicates some questions ([#2124](#2124)) ([ed9bbe6](ed9bbe6)) CHANGELOG CHANGELOG.md CI LICENSE README.md TODO.md apps assets error_pages jest-shim.ts jest.config.ts jest.preset.js libs makefile migration-storybook.log node_modules nx.json package-lock.json package.json release.config.js tailwind.config.js tools tsconfig.base.json incorrect sorting on api configuration ([#2125](#2125)) ([5acf213](5acf213)) CHANGELOG CHANGELOG.md CI LICENSE README.md TODO.md apps assets error_pages jest-shim.ts jest.config.ts jest.preset.js libs makefile migration-storybook.log node_modules nx.json package-lock.json package.json release.config.js tailwind.config.js tools tsconfig.base.json reference data would appear as object object in grid ([b9b1910](b9b1910)) CHANGELOG CHANGELOG.md CI LICENSE README.md TODO.md apps assets error_pages jest-shim.ts jest.config.ts jest.preset.js libs makefile migration-storybook.log node_modules nx.json package-lock.json package.json release.config.js tailwind.config.js tools tsconfig.base.json reset default button not available in front-office ([#2122](#2122)) ([ab81073](ab81073)) CHANGELOG CHANGELOG.md CI LICENSE README.md TODO.md apps assets error_pages jest-shim.ts jest.config.ts jest.preset.js libs makefile migration-storybook.log node_modules nx.json package-lock.json package.json release.config.js tailwind.config.js tools tsconfig.base.json some fields would not be accessible in map popup ([48f98aa](48f98aa)) CHANGELOG CHANGELOG.md CI LICENSE README.md TODO.md apps assets error_pages jest-shim.ts jest.config.ts jest.preset.js libs makefile migration-storybook.log node_modules nx.json package-lock.json package.json release.config.js tailwind.config.js tools tsconfig.base.json sorting in ref data table not working ([#2126](#2126)) ([1ae1e27](1ae1e27)) ### Features CHANGELOG CHANGELOG.md CI LICENSE README.md TODO.md apps assets error_pages jest-shim.ts jest.config.ts jest.preset.js libs makefile migration-storybook.log node_modules nx.json package-lock.json package.json release.config.js tailwind.config.js tools tsconfig.base.json add possibility to fetch nested fields in ref data ([efbcc1d](efbcc1d)) CHANGELOG CHANGELOG.md CI LICENSE README.md TODO.md apps assets error_pages jest-shim.ts jest.config.ts jest.preset.js libs makefile migration-storybook.log node_modules nx.json package-lock.json package.json release.config.js tailwind.config.js tools tsconfig.base.json allow to edit / add reference data fields ([#2128](#2128)) ([c7f047b](c7f047b))
AntoineRelief pushed a commit that referenced this pull request Dec 7, 2023
…a.2...v2.2.0-alpha.3) (2023-12-07) ### Bug Fixes CHANGELOG CHANGELOG.md CI LICENSE README.md TODO.md apps assets error_pages jest-shim.ts jest.config.ts jest.preset.js libs makefile migration-storybook.log node_modules nx.json package-lock.json package.json release.config.js tailwind.config.js tools tsconfig.base.json editing widgets could sometimes scroll to top of dashboard ([#2135](#2135)) ([da6dd93](da6dd93)) CHANGELOG CHANGELOG.md CI LICENSE README.md TODO.md apps assets error_pages jest-shim.ts jest.config.ts jest.preset.js libs makefile migration-storybook.log node_modules nx.json package-lock.json package.json release.config.js tailwind.config.js tools tsconfig.base.json few issues with templating & reference data ([c504e92](c504e92)) CHANGELOG CHANGELOG.md CI LICENSE README.md TODO.md apps assets error_pages jest-shim.ts jest.config.ts jest.preset.js libs makefile migration-storybook.log node_modules nx.json package-lock.json package.json release.config.js tailwind.config.js tools tsconfig.base.json incorrect sorting on api configuration ([#2125](#2125)) ([5acf213](5acf213)) CHANGELOG CHANGELOG.md CI LICENSE README.md TODO.md apps assets error_pages jest-shim.ts jest.config.ts jest.preset.js libs makefile migration-storybook.log node_modules nx.json package-lock.json package.json release.config.js tailwind.config.js tools tsconfig.base.json incorrectly sized columns ([#2127](#2127)) ([c0be7b1](c0be7b1)) CHANGELOG CHANGELOG.md CI LICENSE README.md TODO.md apps assets error_pages jest-shim.ts jest.config.ts jest.preset.js libs makefile migration-storybook.log node_modules nx.json package-lock.json package.json release.config.js tailwind.config.js tools tsconfig.base.json pull jobs would not be editable ([cc9d685](cc9d685)) CHANGELOG CHANGELOG.md CI LICENSE README.md TODO.md apps assets error_pages jest-shim.ts jest.config.ts jest.preset.js libs makefile migration-storybook.log node_modules nx.json package-lock.json package.json release.config.js tailwind.config.js tools tsconfig.base.json remove code information in url after login ([bcfb603](bcfb603)) CHANGELOG CHANGELOG.md CI LICENSE README.md TODO.md apps assets error_pages jest-shim.ts jest.config.ts jest.preset.js libs makefile migration-storybook.log node_modules nx.json package-lock.json package.json release.config.js tailwind.config.js tools tsconfig.base.json sorting in ref data table not working ([#2126](#2126)) ([1ae1e27](1ae1e27)) CHANGELOG CHANGELOG.md CI LICENSE README.md TODO.md apps assets error_pages jest-shim.ts jest.config.ts jest.preset.js libs makefile migration-storybook.log node_modules nx.json package-lock.json package.json release.config.js tailwind.config.js tools tsconfig.base.json text widget edition would lose widget display configuration ([7f3b18b](7f3b18b)) ### Features CHANGELOG CHANGELOG.md CI LICENSE README.md TODO.md apps assets error_pages jest-shim.ts jest.config.ts jest.preset.js libs makefile migration-storybook.log node_modules nx.json package-lock.json package.json release.config.js tailwind.config.js tools tsconfig.base.json allow text & summary card widgets to use record edition when using resource & layout ([#2134](#2134)) ([a0f0ca0](a0f0ca0)) CHANGELOG CHANGELOG.md CI LICENSE README.md TODO.md apps assets error_pages jest-shim.ts jest.config.ts jest.preset.js libs makefile migration-storybook.log node_modules nx.json package-lock.json package.json release.config.js tailwind.config.js tools tsconfig.base.json allow to edit / add reference data fields ([#2128](#2128)) ([c7f047b](c7f047b))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants