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

Release v1.1.0 #1296

Merged
merged 89 commits into from
Jun 23, 2022
Merged

Release v1.1.0 #1296

merged 89 commits into from
Jun 23, 2022

Conversation

louise-davies
Copy link
Member

@louise-davies louise-davies commented Jun 21, 2022

Description

Release v1.1.0. You can see the format of the new CHANGELOG which is generated using github_changelog_generator. Only review the last commit as all the others are just from develop and so have already been reviewed 😅

Testing instructions

Add a set up instructions describing how the reviewer should test the code

  • Review code
  • Check Actions build
  • Review changes to test coverage

dependabot bot and others added 30 commits March 26, 2022 01:50
Bumps [node-forge](https://github.com/digitalbazaar/forge) from 1.2.1 to 1.3.0.
- [Release notes](https://github.com/digitalbazaar/forge/releases)
- [Changelog](https://github.com/digitalbazaar/forge/blob/main/CHANGELOG.md)
- [Commits](digitalbazaar/forge@v1.2.1...v1.3.0)

---
updated-dependencies:
- dependency-name: node-forge
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
- think I mostly have everything working but no tests
Bumps [minimist](https://github.com/substack/minimist) from 1.2.5 to 1.2.6.
- [Release notes](https://github.com/substack/minimist/releases)
- [Commits](https://github.com/substack/minimist/compare/1.2.5...1.2.6)

---
updated-dependencies:
- dependency-name: minimist
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
- also split downloadApi into api functions and rq hooks
- this makes it easier to test and also smaller files
- also add test for disabling download of carts with empty items
- also fixed bug in e2e server code serving the wrong settings file
- also memoise loadMoreRows
- remove batched updates and increase the concurrency
Depending on if a filterByTime prop is passed, the date filter component will return either a KeyboardDatePicker or a KeyboardDateTimePicker. Thus, the admin download table now contains a KeyboardDateTimePicker for more accurate filtering of results
Now completely removing the ability to set the seconds value in the admin download table date filtering. Previously, this was possible and causing issues with expected results. To align with the date picker GUI, we can now only go down to the minute level.

Also removing changes made to index.tsx. This was to mistakenly account for time inputs to be filtered but this is not needed in index as it can only receive a set date value. As such, the start date will carry on with a time of 00:00:00 and the end date will carry on with a time of 23:59:59 inclusive
Increases number of lines so worth deciding if this is beneficial or not
…ate column filter #1175

To align with Topcat, the date pickers in the download table need to be date time pickers

Fixing bug in aria hidden prop passing to date picker objects
Incorrectly "fixed" the test, there is still something going wrong here
…common props

- also extract more common props
…node-forge-1.3.0

Bump node-forge from 1.2.1 to 1.3.0
…minimist-1.2.6

Bump minimist from 1.2.5 to 1.2.6
Refactor download cart to improve performance
louise-davies and others added 23 commits May 16, 2022 13:34
…ix/fix-breadcrumbs-visual-glitch-#1196' of https://github.com/ral-facilities/datagateway into bugfix/fix-breadcrumbs-visual-glitch-#1196
…isual-glitch-#1196

Fix breadcrumbs visual glitch
…ss-#1196

#1196 - Use MUI theme background colour for breadcrumb gaps
- and don't send a notification event, as SG will do it for us if the refresh fails
- also tidy up code in the download button
…-error-#1257

Bugfix/fix token refresh error #1257
…button-#1261

Bugfix/disable download button #1261
Bumps [async](https://github.com/caolan/async) from 2.6.3 to 2.6.4.
- [Release notes](https://github.com/caolan/async/releases)
- [Changelog](https://github.com/caolan/async/blob/v2.6.4/CHANGELOG.md)
- [Commits](caolan/async@v2.6.3...v2.6.4)

---
updated-dependencies:
- dependency-name: async
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
…async-2.6.4

Bump async from 2.6.3 to 2.6.4
@louise-davies louise-davies changed the base branch from develop to main June 21, 2022 13:29
@codecov
Copy link

codecov bot commented Jun 21, 2022

Codecov Report

Merging #1296 (d0feb83) into main (3a5073e) will decrease coverage by 0.51%.
The diff coverage is 95.56%.

@@            Coverage Diff             @@
##             main    #1296      +/-   ##
==========================================
- Coverage   98.01%   97.49%   -0.52%     
==========================================
  Files         134      134              
  Lines        8106     6827    -1279     
  Branches     2437     2055     -382     
==========================================
- Hits         7945     6656    -1289     
- Misses        146      155       +9     
- Partials       15       16       +1     
Flag Coverage Δ
common 98.16% <100.00%> (-0.46%) ⬇️
dataview 97.98% <95.83%> (+0.09%) ⬆️
download 94.96% <94.52%> (-2.33%) ⬇️
search 97.37% <50.00%> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
packages/datagateway-common/src/api/datafiles.tsx 97.67% <ø> (-1.60%) ⬇️
packages/datagateway-common/src/api/datasets.tsx 100.00% <ø> (ø)
...ages/datagateway-common/src/api/facilityCycles.tsx 100.00% <ø> (ø)
packages/datagateway-common/src/api/index.tsx 96.68% <ø> (-1.08%) ⬇️
...ackages/datagateway-common/src/api/instruments.tsx 100.00% <ø> (ø)
...ages/datagateway-common/src/api/investigations.tsx 100.00% <ø> (ø)
packages/datagateway-common/src/api/lucene.tsx 100.00% <ø> (ø)
packages/datagateway-common/src/api/studies.tsx 100.00% <ø> (ø)
packages/datagateway-common/src/app.types.tsx 100.00% <ø> (ø)
...ateway-dataview/src/page/breadcrumbs.component.tsx 95.60% <ø> (-1.11%) ⬇️
... and 70 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3a5073e...d0feb83. Read the comment docs.

Copy link
Contributor

@sam-glendenning sam-glendenning left a comment

Choose a reason for hiding this comment

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

Looks good. Can't see any references to old version numbers

@louise-davies louise-davies merged commit 7324fc0 into main Jun 23, 2022
@louise-davies louise-davies deleted the release/v1.1.0 branch June 23, 2022 10:04
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.

3 participants