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

Mixed styles of note and tip in docu #1231

Merged
merged 7 commits into from
Apr 8, 2020
Merged
Show file tree
Hide file tree
Changes from 4 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 1 addition & 3 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
# Luigi
<p align="center">
<img src="https://raw.githubusercontent.com/sap/luigi/master/logo.png" width="235">
</p>
</p>

## Overview

Expand Down Expand Up @@ -57,7 +57,6 @@ import {
} from '@luigi-project/client-ie11';
```

<!-- add-attribute:class:warning -->
> **NOTE**: The example applications are not fully compatible with IE11.

## Development
Expand All @@ -72,7 +71,6 @@ For security reasons, follow these guidelines when developing a micro frontend:
- Maintain a whitelist with trusted domains and compare it with the origin of the Luigi Core application. The origin will be passed when you call the init listener in your micro frontend. Stop further processing if the origin does not match.


<!-- add-attribute:class:success -->
> **NOTE**: Luigi follows these [sandbox rules for iframes](https://github.com/SAP/luigi/blob/af1deebb392dcec6490f72576e32eb5853a894bc/core/src/utilities/helpers/iframe-helpers.js#L140).


Expand Down
2 changes: 1 addition & 1 deletion docs/advanced-scenarios.md
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ This example shows you how to keep an existing routing strategy and use an exist

1. Configure the Luigi navigation node:

<!-- add-attribute:class:success -->
<!-- add-attribute:class:warning -->
> **NOTE**: To keep the example simple, we use [virtualTree](navigation-parameters-reference.md#virtualtree) to allow any nested navigation, but this is not mandatory. You can always specify the node tree yourself and still use automatic navigation with router events.

```javascript
Expand Down
4 changes: 2 additions & 2 deletions docs/authorization-configuration.md
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@ anonymousAccess: true // always show nodes

## OpenID Connect configuration

This code snippet demonstrates how to configure authorization using OpenID Connect in Luigi. Note that you must install the [OpenID Plugin](https://github.com/SAP/luigi/tree/master/plugins/auth/public/auth-oidc) first.
This code snippet demonstrates how to configure authorization using OpenID Connect in Luigi. Note that you must install the [OpenID Plugin](https://github.com/SAP/luigi/tree/master/plugins/auth/public/auth-oidc) first.

```javascript
import oidcProvider from '@luigi-project/plugin-auth-oidc';
Expand Down Expand Up @@ -223,7 +223,7 @@ After authorization is successful on the authorization provider's side, it redir
[openIdConnect.js](../core/src/providers/auth/openIdConnect.js) lazy loads the official `oidc-client` library and is a good starting point if you also depend on external authorization libraries.

<!-- add-attribute:class:success -->
<!-- add-attribute:class:warning -->
> **NOTE:** Read more about authorization helpers in the [Core API: AuthorizationStore](luigi-core-api.md#AuthorizationStore) section.
### Persisting auth data
Expand Down
1 change: 1 addition & 0 deletions docs/navigation-advanced.md
Original file line number Diff line number Diff line change
Expand Up @@ -214,6 +214,7 @@ The purpose of contexts is to send objects to the micro frontend. You can do thi
- **type**: object
- **description**: sends the specified object as context to the view. Use this property in combination with the dynamic **pathSegment** to receive the context through the context listeners of Luigi Client. This is an alternative to using the dynamic value in the **viewUrl**.

<!-- add-attribute:class:warning -->
> **NOTE**: Context should not be used to create the path or URL as this can lead to errors. Please use one of the methods described in the [dynamically changeable paths](#dynamically-changeable-paths) section instead.


Expand Down
13 changes: 6 additions & 7 deletions test/e2e-test-application/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@

This is the Angular-based sample application which runs with Luigi framework.

<!-- add-attribute:class:success -->
> **NOTE:** The authorization flow `mockAuth` in this application is a mock implementation for local development. **DO NOT USE IN PRODUCTION!**

## Development
Expand Down Expand Up @@ -40,8 +39,8 @@ To have this application running, follow these steps:

- To run Luigi with hash-based routing, set the **routing.useHashrouting** configuration to `true`.
- To run Luigi with path-based routing, set the **routing.useHashrouting** configuration to `false`.
Run the server with the following command:

Run the server with the following command:
```bash
npm run start
```
Expand All @@ -50,13 +49,13 @@ To have this application running, follow these steps:

6. Optional: Reflect the changes introduced to the Luigi Core code in the sample application.

- Allow the Luigi Core to bundle every change you apply to it. The easiest approach is to open the Luigi `root` folder in another tab of your terminal window and run the following command:
```bash
- Allow the Luigi Core to bundle every change you apply to it. The easiest approach is to open the Luigi `root` folder in another tab of your terminal window and run the following command:

```bash
lerna run bundle-develop
```
- The Luigi Client is not bundled, so you can update it without bundling.

- The auto-reload of your application updates only the application. Type `CMD + R` to reflect the changes in the linked Luigi Core and Luigi Client modules on the website.


Expand Down