forked from DSpace/dspace-angular
-
Notifications
You must be signed in to change notification settings - Fork 1
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
Edit user page - added missing info #626
Open
milanmajchrak
wants to merge
310
commits into
dtq-dev
Choose a base branch
from
ufal/user-edit-page
base: dtq-dev
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.
Open
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
* Initial commit * updated with google scholar metadata from google-scholar.properties * modified metadata.ts and tests * reverted wrong changes Co-authored-by: MilanMajchrák <[email protected]> Co-authored-by: MajoBerger <[email protected]>
* Init commit * Added '/admin/handle' icon to the admin menu and created tests. * Update handle menu option test * Redirect to some page after click on 'handle' menu option * After click on menu option 'handle' is redirect to the /handle page * The handle table shows some handles but response is too slow * Added loading component * Removed DSpaceObject from handle.model.ts because converting DSpaceObject takes too long. * handle data wasn't actual after changing page in pagination - fixed * Added handle operation buttons with routing and messages. * Created redirection to the new handle and edit handle page * send edit handle parameters to the edit handle page * Edit and delete works * Disable edit and delete button if none handle is selected. * Refresh the table after handle deletion. * redirecting doesn't work * finally it redirect correctly * Fixed subscription error * New handle page created * Handle page graphics * Created page and routing for changing handle prefix * Added input validation to the change handle prefix form. * some little refactoring * Create IT test for checking the handle page and UT for the ChangePrefixComponent * Create IT test for checking the handle page and UT for the ChangePrefixComponent * Created test for the edit-handle page component * Created tests for the NewHandlePageComponent and handle-table-page and handle-global-actions * Created sample tests for the HandleTable component * Created tests for the HandleTableComponent. * added handle url to the request * Added notifications * Updated edit handle page - resource type is showed and added solrProperties to the pagination. * fixed some errors * Added filtering the handles in the handle-table * Some refactoring * refactoring * Strange error has occured * The tests fixed * Fixed the integration test - admin-menu * Added comments for the empty *.scss files and updated messages in the en.json5 file * Added comments for the empty *.scss file Co-authored-by: MilanMajchrák <[email protected]>
…t:8080' not dtq-dev5.pc * Fixed production error: the admin cannot create the new handle because it throws Unknown Error. Reason: the request was sent to the 'localhost:8080' not server url, that url was hardcoded. * fix failing test in the community-list.spec.ts Co-authored-by: MilanMajchrák <[email protected]>
…ge tables is not seen for user
* added data import thru python lib * deleted logs * stupid \\:
* feature/lf-3-license-selector-dtq-lf (#103) design changes, clarin licenses changed to two steps * Initial commit * styles for license-selector * added license selector * added yarn * temp commit - work on progress * the license show validation errors * licenses 4 license selector is loaded from definition * Added messages to the license step page * working on selecting license definitoin * Select the license and assign it to the attribute value * working on sending clarin license from license selector to the BE * Show not supported clarin license error * Loaded the license after refresh * Added section status * Trying to add distribution license satic page * Added static page distribution licenses * added contract breadcrumbs * Trying to send the accepting distribution license to the BE * working on clarin license distribution step * Distribution license agreement is in the separated step * contract page license data is loaded dynamically * fixed choosing the resource licenses * some refactoring and design changes * some refactoring and design changes * some design changes * some design changes * strange tast failure fixing - temp commit * test github actions if wont fail * Added test classes * Fixed error after fixing conflicts * Added license selector css * trying of fixing 'cannot find license-selector.js' error Co-authored-by: MilanMajchrák <[email protected]> Co-authored-by: MajoBerger <[email protected]> * Feature/lf 3 license selector (#105) * Initial commit * styles for license-selector * added license selector * added yarn * temp commit - work on progress * the license show validation errors * licenses 4 license selector is loaded from definition * Added messages to the license step page * working on selecting license definitoin * Select the license and assign it to the attribute value * working on sending clarin license from license selector to the BE * Show not supported clarin license error * Loaded the license after refresh * Added section status * Trying to add distribution license satic page * Added static page distribution licenses * added contract breadcrumbs * Trying to send the accepting distribution license to the BE * working on clarin license distribution step * Distribution license agreement is in the separated step * contract page license data is loaded dynamically * fixed choosing the resource licenses * some refactoring and design changes * some refactoring and design changes * some design changes * some design changes * strange tast failure fixing - temp commit * test github actions if wont fail * Added test classes * Fixed error after fixing conflicts * Added license selector css * trying of fixing 'cannot find license-selector.js' error Co-authored-by: MilanMajchrák <[email protected]> Co-authored-by: MajoBerger <[email protected]> * Feature/lf 4 license administrator (#106) * Initial commit * Added `License Administration` to the menu * created clarin-license-page.component * Create clarin license label component * Show clarin licenses in the table with clarin license labels * Created buttons * Created define-license-form * Define a new clarin license works * Checked Extended Clarin License Labels and working on checking non extended cll * Cannot edit extended license labels * Define License component is used for updating the License * Editing of licenses works * Send the icon as byte array to the BE * Showed clarin license label image * some functionalities of the license admin should work * created some tests - admin menu and clarin-license-table * created tests for clarin-license-table.component * Created tests for define license and define license label * revert useless changes * some refactoring * Added some docs and refactoring * Added docs and refactoring * Fixed failing tests * Fixed community-list.spec.ts IT * revert wrong fix * Added user requirement to the Clarin License * Fixed parsing clarin license require info Co-authored-by: MilanMajchrák <[email protected]> * fixed error with wrong row * Update docker.yml * Update docker.yml * Update deploy.yml * Update deploy.yml added importing licenses to the deploy action * fixed license selector duplication * Internal/presentation customize home page (#107) Co-authored-by: MajoBerger <[email protected]> Co-authored-by: MilanMajchrák <[email protected]> * fixed breadcrumbs top margin * some desing chhanges - button colors * Update deploy.yml do not import licenses * fixed license table desing and home page modification resizing * Added language icons to the navbar and changed graph percentages * Added login and logout layout to the navbar. * added data import thru python * separate lib contents in action writeout * changed \\ to / * dont checkout submodules in action * false * git not ssh * Changed license table column width * Changed clarin license checkbox width * eF u git submodules * The user is redirected to the community-list after clicking on `catalogue` on menu * Update deploy.yml remove data import because deploy is failing * feature/lf-1-license-visible-in-item-view added license info * Initial commit * Added clarin-license-info component and changed colors * License info is showed in the item view * Added license info to the full-page * Fixed unit tests Co-authored-by: MilanMajchrák <[email protected]> * Added search filter messages * Do not build a new docker image on commit * Some refactoring and added unit tests for the license-contract-page * fixed failing tests in the define-license-form.component.spec.ts * Copied login function from upstream * Added Integration Tests for the License steps and changed login process. * Changed TEST_ENTITY_PUBLICATION * changed FALLBACK_TEST_REST_BASE_URL to dev5/server * Changed login command to ui clicking version * Commented accessibility tests * Fixed login process * Done some refactoring and added some docs. * Done some refactoring and added some docs. * Trying to fix failing integration tests. * Trying to fix failing submission-ui.spec.ts integration tests. * Fixed integration tests test collection uuid and fixed login process. * feature/lf-2-support-4-restrictive-licenses (#96) bitstream downloads * Initial commit * Created bitstream download management * Added some comments * Updated redirecting based on the authorization. * Loaded License Agreement data * Created license agreement page without sending user information * Get all user metadata * Show required metadata fields with data from database. * Loaded userMetadata for not singed in and signed in user. * Added token expiration message and finished downloading of the bitstream * Fixed license errors. Co-authored-by: MilanMajchrák <[email protected]> * done little refactoring * fixed (commented) tests * fixed error * fixed lint errors * The License Label with extended = true was transformed to extended = false in the response of the method findAll. I defined response of findAll method in the define-license-form.component.spec.ts, previously it was loaded from clarin-license-mock.ts * Fixed integration tests - added more tries. * Added more tries for tombstone IT Co-authored-by: MilanMajchrák <[email protected]> Co-authored-by: MajoBerger <[email protected]> Co-authored-by: MajoBerger <[email protected]>
* Initial commit * Fixed problem with reloading the item version. Problem was that the method `showSubmitter` was still called. * Fixed lint issue Co-authored-by: MilanMajchrák <[email protected]>
* Initial commit * Added discojuice, aai, aai config to the login * Added CSS to the discojuice * Done aai for now * Fixed linting * Fixed unit tests * Little refactoring * Created component for missing-idp headers - static error page, for sending the verification token to the BE, for filling the verificatin token. * Created component for missing-idp headers - static error page, for sending the verification token to the BE, for filling the verificatin token. * Revert "Created component for missing-idp headers - static error page, for sending the verification token to the BE, for filling the verificatin token." This reverts commit 7b59737. * Revert "Created component for missing-idp headers - static error page, for sending the verification token to the BE, for filling the verificatin token." This reverts commit adb264f. Co-authored-by: MilanMajchrák <[email protected]>
* Initial commit * Added discojuice, aai, aai config to the login * Added CSS to the discojuice * Done aai for now * Fixed linting * Fixed unit tests * Little refactoring * Created component for missing-idp headers - static error page, for sending the verification token to the BE, for filling the verificatin token. * Created component for missing-idp headers - static error page, for sending the verification token to the BE, for filling the verificatin token. * User is logged automatically. * Small refactoring * Small refactoring * Small refactoring and added docs * Commented tests * Added messages to the autoregistration component * Removed test files which doesn't have tests. Co-authored-by: MilanMajchrák <[email protected]>
* Initial commit * Temp commit - working on refbox ui * Added shared buttons * Fixed dropdown button issue - added bootstrap scripts to the scripts in the `angular.json` * Added featured services dropdown buttons * The item text citation added * Added tooltip - copied * The item is possible to share on social media * Added parsing of featured services and rendering for buttons. * Temp commit - working on the copying modal content * The OAI metadata are showed in the citation modal * Beautify ref box * Added docs and messages * Commented accessibility tests * Removed test files which doesn't have tests. Co-authored-by: MilanMajchrák <[email protected]>
The parameter -o (organization) is required.
Co-authored-by: MilanMajchrák <[email protected]>
* Initial commit * Added the statistics button to the item-page and after clicking on the the user is redirected to the `matomo-statistics` page * Added example graph to the component. * Show year and month data. * Get totalData views * View day data * Styled views and downloads buttons * Changed chart colors * Created chart sorting * Show count of downloaded file * Finished the clarin matomo statistics - the data are loaded from the cache-server. * Removed test files --------- Co-authored-by: MilanMajchrák <[email protected]>
* Added carousel to the top of the home-page. * Refactored carousel * Updated carousel for mobile devices * Designed Search component * Added Subject, Author, Language searching fields * Created component for showing the item in the box * Created item-view-box * Show most viewed items very slowly * Almost done The most viewed Items. * Added authors to the item-box * Show every author * Detailing of the home-page * Detailing of the home-page * Added community indexation * Trying to fix failing tests * Fixed failing test and added docs to the home-page * Added docs * Added the clarin item box into search results page. * Revert vanilla changes --------- Co-authored-by: MilanMajchrák <[email protected]>
…#133) * Added a new options to the sidebar menu. * Fixed error - id of the menu component cannot contains `statistics` * Fixed login - logout div - it was overlaying by the menu. * Fixed background color for the mobile devices/ * Selecting of the language works. --------- Co-authored-by: MilanMajchrák <[email protected]>
) Co-authored-by: MilanMajchrák <[email protected]>
* Initial commit * Added maximum length in the complex input type. --------- Co-authored-by: MilanMajchrák <[email protected]>
…larin-search` * Initial commit * Added maximum length in the complex input type. * Added `clarin-search` component to the search-page --------- Co-authored-by: MilanMajchrák <[email protected]>
Co-authored-by: MilanMajchrák <[email protected]>
…info * Fixed loading the license into selection and fixed assigning wrong license after choosing it. * Fixed sending an email with download link - added email input field. * Removed console log messages. --------- Co-authored-by: MilanMajchrák <[email protected]>
* Trying to compose required information to the item page. * Raw version of the updated item-page. * Fixed ref-box when the item has more authors. * Removed unused files. * Removed logging to the console. * Export new component from the item-page.module * Refactoring - beautify the code * Removed disabled components from the item-page.module * Removed unuseful tests. --------- Co-authored-by: MilanMajchrák <[email protected]>
* Updated browsing links to do not overflow to the next line * Add truncate length to some constant
1. Show publisher in the refbox if the Item has no authors. (#599) 2. Revert changes from the PR where the `:uknow author` was hidden - this feature is more complex and will be developed with the configurable ref box issue.
- local.cfg has passwords etc. - ui url is used when minting dois - logging for cli
…ed required info (#595) * Load max 100 license labels in the edit license page * The icon is not required during creation of the clarin license label * Updated creating of the clarin license label - refactored code, the icon is not required * Fixed issue with wrong loaded required info. * Updated test which controlles creating license label without icon * Replaced `if` condition by `catch`.
* Removed static files with license definitions. They are not already used because license definitions are loaded from the BE. * Updated download cmd command and removed copy button * Updated `lindat.mff.cuni.cz/repository/xoai/..` with `lindat.mff.cuni.cz/repository/static` * Removed hardcoded lindat url from static pages.
…er metadata into request. (#611)
* Added items/download/zip path for the email download link. * Add dtoken to the download request because of authorization. * Call the right download request if the user downloads ZIP. * Redirect to download/zip page for authorization when the user is downloading the ZIP
* Added authorization and basic redirect to the `Edit Item - License` component * Show current license * Added a selector for licenses * Added `Update` and `Return` buttons and all html messages into EN language. * Send the request to the BE endpoint with a new licenseId * The current license is updated after the request is successfull. Added notification. Added redirect for defining a new license. * Translated messages to cs * Updated test file for passing tests * Translation `zmáčkněte změnit` changed to `zmáčkněte aktualizovat`.
* Show EU funding name instead of organization name. * Show required character in the submission complex input field `*` however the whole complex input field is not required. * Refactored adding of error messages into complex input field. * Mark nested input field from the complex input field as required, however the complex input field is not required. * Removed complex input field when the value is `N/A`
#621) [Port dspace-7_x] Fix missing Register DOI button if configuration enabled Co-authored-by: Tim Donohue <[email protected]>
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.
PR: #467