Skip to content

fix(related-data): sequelize related data are not displayed#772

Merged
romain-gilliotte merged 4 commits intomasterfrom
fix/broken-sequelize-has-many
Jul 22, 2021
Merged

fix(related-data): sequelize related data are not displayed#772
romain-gilliotte merged 4 commits intomasterfrom
fix/broken-sequelize-has-many

Conversation

@romain-gilliotte
Copy link
Copy Markdown
Contributor

@romain-gilliotte romain-gilliotte commented Jul 21, 2021

No description provided.

… by URL does not display any records on mongoose (#758)"

This reverts commit 9cfae72.
@romain-gilliotte romain-gilliotte changed the title Revert "fix(related-data): reaching an embedded has-many relationship by URL does not display any records on mongoose (#758)" fix(related-data): revert reaching an embedded has-many relationship by URL does not display any records on mongoose (#758) Jul 21, 2021
@romain-gilliotte romain-gilliotte changed the title fix(related-data): revert reaching an embedded has-many relationship by URL does not display any records on mongoose (#758) fix(related-data): sequelize related data are not displayed Jul 22, 2021
@qlty-cloud-legacy
Copy link
Copy Markdown

Code Climate has analyzed commit 8f1a4ff and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (60% is the threshold).

This pull request will bring the total coverage in the repository to 56.0%.

View more on Code Climate.

Copy link
Copy Markdown
Contributor

@adriguy adriguy left a comment

Choose a reason for hiding this comment

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

🤖

@adriguy adriguy assigned romain-gilliotte and unassigned adriguy Jul 22, 2021
@romain-gilliotte romain-gilliotte merged commit 27008e9 into master Jul 22, 2021
@romain-gilliotte romain-gilliotte deleted the fix/broken-sequelize-has-many branch July 22, 2021 09:30
forest-bot added a commit that referenced this pull request Jul 22, 2021
## [9.0.1](v9.0.0...v9.0.1) (2021-07-22)

### Bug Fixes

* **related-data:** sequelize related data are not displayed ([#772](#772)) ([27008e9](27008e9))
@forest-bot
Copy link
Copy Markdown
Member

🎉 This PR is included in version 9.0.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

ShohanRahman pushed a commit that referenced this pull request Jan 25, 2022
ShohanRahman pushed a commit that referenced this pull request Jan 25, 2022
# [8.0.0](ForestAdmin/forest-express-sequelize@v7.12.2...v8.0.0) (2021-07-19)

### Bug Fixes

* abstractRecord type ([#772](ForestAdmin/forest-express-sequelize#772)) ([81492b9](ForestAdmin/forest-express-sequelize@81492b9))
* correctly get the IP address from the request headers ([2b5ed1d](ForestAdmin/forest-express-sequelize@2b5ed1d))
* fix types ([580ef41](ForestAdmin/forest-express-sequelize@580ef41))

### Features

* allow hooks for global and bulk smart action ([#769](ForestAdmin/forest-express-sequelize#769)) ([14c26cd](ForestAdmin/forest-express-sequelize@14c26cd))
* **browsing-context:** allow `Forest-Context-Url` header to give the current browser url of users ([#665](ForestAdmin/forest-express-sequelize#665)) ([c46fd66](ForestAdmin/forest-express-sequelize@c46fd66))
* **browsing-context:** allow `Forest-Context-Url` header to give the current browser url of users ([#665](ForestAdmin/forest-express-sequelize#665)) ([a61699a](ForestAdmin/forest-express-sequelize@a61699a))
* **security:** secure segments queries ([#764](ForestAdmin/forest-express-sequelize#764)) ([98d7f74](ForestAdmin/forest-express-sequelize@98d7f74))
* **smart-action:** user can add dynamically field with hook on smart action form ([#742](ForestAdmin/forest-express-sequelize#742)) ([228bb42](ForestAdmin/forest-express-sequelize@228bb42))

* feat(scopes)!: enforce scopes restrictions on a wider range of requests (#731) ([da7d71d](ForestAdmin/forest-express-sequelize@da7d71d)), closes [#731](ForestAdmin/forest-express-sequelize#731)

### BREAKING CHANGES

* record is no longer send to the hook middleware & values option on smart action is no longer supported
* **smart-action:** fields parameters on hook function is no longer a map of field, it is now an array.
change hook is no longer choosen by the field name, field need to have hook defined inside it definition by addin a props hook.
* the public API of forest-express has changed to include the parameters which are needed to evaluate scope filters. This includes PermissionMiddlewareCreator, all classes on the /exposed folder and most services. Many routes now require providing the current user timezone.
* **browsing-context:** users willing to use this header needs either to clean the allowed headers or to add this specific header in the CORS configuration
* **browsing-context:** users willing to use this header needs either to clean the allowed headers or to add this specific header in the CORS configuration
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants