[v4.6.0] It seems that you should use `??` Instead of `| |` about notification

[afc163]

I checkout a new branch for 4.x to maintain: https://github.com/react-component/notification/tree/4.x

[kiner-tang]

I want to do it, but which branch should I develop on? It appears that the '5.x' version refactored this component. The problem is currently in version '4.x'！@zombieJ

[afc163]

master

[kiner-tang]

master

The master branch should be '5.x' code, this version of the code does not have this problem, this problem should only appear in '4.x', should we upgrade the 'rc-notification' version to '5.0.0' in 'ant-design'? Or do we need a patch fix for version 4.x

notification/src/useNotification.tsx

Line 106 in fc89563

if (mergedConfig.key === null || mergedConfig.key === undefined) {

[afc163]

a patch fix for version 4.x

Just send PR to master branch.

[kiner-tang]

a patch fix for version 4.x

Just send PR to master branch.

But the code in the 'master' branch has been refactored, there is no '4.x' related problem code, in the branch list, only '3.x' and 'master' to choose from, which branch should I use as a benchmark for development?

[HelloBojack]

mater 代码已经是 5.0-alpha 版本了，没有这个问题了，这个问题出现在 4.6 之前版本，如果 antd 后面更新 rc-notification 到最新版，就没有问题了。所以这个bug，是不是 antd 等更新，不用修复了？

[kiner-tang]

I checkout a new branch for 4.x to maintain: https://github.com/react-component/notification/tree/4.x

Great. I can fix it now

[afc163]

https://github.com/react-component/notification/tree/v4.6.1