Skip to content

Commit

Permalink
Merge branch 'dev'
Browse files Browse the repository at this point in the history
  • Loading branch information
michaelmaillot committed May 29, 2024
2 parents 0f857b9 + f4cc5ae commit bf61b7a
Show file tree
Hide file tree
Showing 8 changed files with 45 additions and 9 deletions.
15 changes: 15 additions & 0 deletions CHANGELOG.json
Original file line number Diff line number Diff line change
@@ -1,5 +1,20 @@
{
"versions": [
{
"version": "3.18.1",
"changes": {
"new": [],
"enhancements": [],
"fixes": [
"`FilePicker`: Fix issue with adding link by typing in 'From a link' tab [#1814](https://github.com/pnp/sp-dev-fx-controls-react/pull/1814)",
"Update nl-nl.ts [#1823](https://github.com/pnp/sp-dev-fx-controls-react/pull/1823)"
]
},
"contributions": [
"[Antanina Druzhkina](https://github.com/Ateina)",
"[Elio Struyf](https://github.com/estruyf)"
]
},
{
"version": "3.18.0",
"changes": {
Expand Down
11 changes: 11 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,16 @@
# Releases

## 3.18.1

### Fixes

- `FilePicker`: Fix issue with adding link by typing in 'From a link' tab [#1814](https://github.com/pnp/sp-dev-fx-controls-react/pull/1814)
- Update nl-nl.ts [#1823](https://github.com/pnp/sp-dev-fx-controls-react/pull/1823)

### Contributors

Special thanks to our contributors (in alphabetical order): [Antanina Druzhkina](https://github.com/Ateina), [Elio Struyf](https://github.com/estruyf).

## 3.18.0

### Enhancements
Expand Down
11 changes: 11 additions & 0 deletions docs/documentation/docs/about/release-notes.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,16 @@
# Releases

## 3.18.1

### Fixes

- `FilePicker`: Fix issue with adding link by typing in 'From a link' tab [#1814](https://github.com/pnp/sp-dev-fx-controls-react/pull/1814)
- Update nl-nl.ts [#1823](https://github.com/pnp/sp-dev-fx-controls-react/pull/1823)

### Contributors

Special thanks to our contributors (in alphabetical order): [Antanina Druzhkina](https://github.com/Ateina), [Elio Struyf](https://github.com/estruyf).

## 3.18.0

### Enhancements
Expand Down
4 changes: 2 additions & 2 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
{
"name": "@pnp/spfx-controls-react",
"description": "Reusable React controls for SharePoint Framework solutions",
"version": "3.18.0",
"version": "3.18.1",
"node": ">=16.13.0 <17.0.0 || >=18.17.1 <19.0.0",
"scripts": {
"build": "gulp build",
Expand Down
2 changes: 1 addition & 1 deletion src/common/telemetry/version.ts
Original file line number Diff line number Diff line change
@@ -1 +1 @@
export const version: string = "3.18.0";
export const version: string = "3.18.1";
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,6 @@ export default class LinkFilePickerTab extends React.Component<ILinkFilePickerTa
className={styles.linkTextField}
label={strings.LinkFileInstructions}
ariaLabel={strings.LinkFileInstructions}
defaultValue={"https://"}
onGetErrorMessage={(value: string) => this._getErrorMessagePromise(value)}
autoAdjustHeight={false}
underlined={false}
Expand Down Expand Up @@ -64,7 +63,7 @@ export default class LinkFilePickerTab extends React.Component<ILinkFilePickerTa
* Called as user types in a new value
*/
private _handleChange = (fileUrl: string): void => {
const filePickerResult: IFilePickerResult = fileUrl && this._isUrl(fileUrl) ? {
const filePickerResult: IFilePickerResult = fileUrl ? {
fileAbsoluteUrl: fileUrl,
fileName: GeneralHelper.getFileNameFromUrl(fileUrl),
fileNameWithoutExtension: GeneralHelper.getFileNameWithoutExtension(fileUrl),
Expand All @@ -81,15 +80,15 @@ export default class LinkFilePickerTab extends React.Component<ILinkFilePickerTa
*/
private _getErrorMessagePromise = async (value: string): Promise<string> => {
// DOn't give an error for blank or placeholder value, but don't make it a valid entry either
if (value === undefined || value === 'https://') {
if (value === undefined || value === '') {
this.setState({ isValid: false });
return '';
}

// Make sure that user is typing a valid URL format
if (!this._isUrl(value)) {
this.setState({ isValid: false });
return '';
return strings.InvalidUrlError;
}

// If we don't allow external links, verify that we're in the same domain
Expand Down
2 changes: 1 addition & 1 deletion src/loc/nl-nl.ts
Original file line number Diff line number Diff line change
Expand Up @@ -145,7 +145,7 @@ define([], () => {
"HighlightColorTeal": "Taling",
"HighlightColorDarkgreen": "Donkergroen",
"HighlightColorPurple": "Paarse",
"HighlightColorMaroon": "Bosneger",
"HighlightColorMaroon": "Kastanjebruin",
"HighlightColorGold": "Goud",
"HighlightColorDarkgrey": "Donkergrijs",
"HighlightColorGrey": "Grijs",
Expand Down

0 comments on commit bf61b7a

Please sign in to comment.