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

Feature/add bulk contentful upload app #26

Open
wants to merge 23 commits into
base: main
Choose a base branch
from

Conversation

DanielClarkeEducation
Copy link
Contributor

@DanielClarkeEducation DanielClarkeEducation commented Feb 16, 2024

Description

POC for how we are going to populate and allow uploads to contentful.
The ticket includes:

  • a .Net console app that will set up contentful with the qualification model and then add entries from a .csv
  • a React app that will be accessible direct from the contentful space to upload .csv data to also add entries

Ticket number (if applicable)

EYQB-13

How Has This Been Tested?

Both apps have manually been tested and shown to work

Screenshots

image

Checklist:

  • My code follows the standards used within this project
  • I have performed a self-review of my code
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests (Unit, E2E) that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

@DanielClarkeEducation DanielClarkeEducation marked this pull request as ready for review February 26, 2024 14:18
@sam-c-dfe sam-c-dfe force-pushed the feature/add-bulk-contentful-upload-app branch from 6888213 to 7e12236 Compare March 6, 2024 15:36
sam-c-dfe and others added 12 commits March 6, 2024 16:32
Bumps [azure/login](https://github.com/azure/login) from 1 to 2.
- [Release notes](https://github.com/azure/login/releases)
- [Commits](Azure/login@v1...v2)

---
updated-dependencies:
- dependency-name: azure/login
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Initial commit of the QualificationsDetails controller & view.

* EYQB-110: Updated to add in unit tests for the controller. Refactored the content service slightly to use a generic method for retrieving the content.

* EYQB-110: Updated to encode qualificationId

* EYQB-110: Updated the view to add in the additional qualification data

* EYQB-110: Added in a temporary page to link to the qualification details for a number of hardcoded qualificationIds

* Added in the qualification details e2e check

* Removed as not needed at this point

* Added a readme file to say how to run the e2e tests locally.

* EYQB-110: Testing the urls config to see if it picks up the new page as a relative path

* EYQB-110: Added absolute paths
Bumps [coverlet.collector](https://github.com/coverlet-coverage/coverlet) from 6.0.1 to 6.0.2.
- [Release notes](https://github.com/coverlet-coverage/coverlet/releases)
- [Commits](coverlet-coverage/coverlet@v6.0.1...v6.0.2)

---
updated-dependencies:
- dependency-name: coverlet.collector
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [contentful.csharp](https://github.com/contentful/contentful.net) from 7.5.0 to 7.5.1.
- [Release notes](https://github.com/contentful/contentful.net/releases)
- [Changelog](https://github.com/contentful/contentful.net/blob/master/CHANGELOG.md)
- [Commits](contentful/contentful.net@v7.5.0...v.7.5.1)

---
updated-dependencies:
- dependency-name: contentful.csharp
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [contentful.aspnetcore](https://github.com/contentful/contentful.net) from 7.5.0 to 7.5.1.
- [Release notes](https://github.com/contentful/contentful.net/releases)
- [Changelog](https://github.com/contentful/contentful.net/blob/master/CHANGELOG.md)
- [Commits](contentful/contentful.net@v7.5.0...v.7.5.1)

---
updated-dependencies:
- dependency-name: contentful.aspnetcore
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* EYQB: Architectural Decision Logs

* EYQB: Typo
* EYQB-133: Initial commit of the content structure for the Start Page. Added custom renderers.

* EYQB-133: Added unit tests and start of the e2e tests

* EYQB-133: Added custom paragraph renderer

* EYQB-133: Removed pargraph renderer and wrapped the content instead

* EYQB-133: Updated to set the body styling on the qualification details page
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